make return values consistent for old and new cutting methods to help…

… isolate cut failures

fix #66

__init__.py

@@ -1839,9 +1839,15 @@ def undo_action(self):
 
 
     def invoke(self, context, event):
+        #HINT you are in contours code
         #TODO Settings harmon CODE REVIEW
         settings = context.user_preferences.addons[AL.FolderName].preferences
 
+        if context.space_data.viewport_shade in {'WIREFRAME','BOUNDBOX'}:
+            self.report({'ERROR'}, 'Viewport shading must be at lease SOLID')
+            return {'CANCELLED'}
+
+
         self.valid_cut_inds = []
         self.existing_loops = []
 
@@ -1899,10 +1905,10 @@ def invoke(self, context, event):
             if len(ed_inds):
                 vert_loops = contour_utilities.edge_loops_from_bmedges(self.dest_bme, ed_inds)
 
-                
+
 
                 if len(vert_loops) > 1:
-                    self.report('WARNING', 'Only one edge loop will be used for extension')
+                    self.report({'WARNING'}, 'Only one edge loop will be used for extension')
                 print('there are %i edge loops selected' % len(vert_loops))
 
                 #for loop in vert_loops:

contour_utilities.py

@@ -2184,7 +2184,7 @@ def cross_section_seed_ver0(bme, mx,
 
     if not len(seeds):
         print('cancelling until your programmer gets smarter')
-        return None
+        return (None,None)
 
     #we have found one edge that crosses, now, baring any terrible disconnections in the mesh,
     #we traverse through the link faces, wandering our way through....removing edges from our list
@@ -2261,7 +2261,7 @@ def cross_section_seed_ver0(bme, mx,
 
         return (verts, eds)
     else:
-        return None
+        return (None, None)