Add mesh extra objects: Update to version 0.3.1

General Pep8 cleanup
Removed unused variables and imports
Removed a panel from add_empty_as_parent
Standardized the property definitions
across all the scripts
Moved scene props from third_domes_panel_271
to init for proper removal
Added a Enum prop for mesh type in teapot
Fixed a small issue with Geodesic domes self.reports
(problem with value fields message spam)
Fixed props names in Geodesic domes
Consistent tooltips
Reorganized menus:
Mechanical Menu including Pipe joints, Mesh gear
Added separators

1 files changed, 67 insertions, 53 deletions

diff --git a/add_mesh_extra_objects/add_empty_as_parent.py b/add_mesh_extra_objects/add_empty_as_parent.py
index b7431b6d..bcaecf64 100644
--- a/add_mesh_extra_objects/add_empty_as_parent.py
+++ b/add_mesh_extra_objects/add_empty_as_parent.py
@@ -1,26 +1,49 @@
 # GPL # Original Author Liero #
 
 import bpy
-from bpy.props import StringProperty, BoolProperty, EnumProperty
+from bpy.types import Operator
+from bpy.props import (
+        StringProperty,
+        BoolProperty,
+        EnumProperty,
+        )
 
-def centro(sel):
-    x = sum([obj.location[0] for obj in sel])/len(sel)
-    y = sum([obj.location[1] for obj in sel])/len(sel)
-    z = sum([obj.location[2] for obj in sel])/len(sel)
-    return (x,y,z)
-
-
-class P2E(bpy.types.Operator):
-    bl_idname = 'object.parent_to_empty'
-    bl_label = 'Parent to Empty'
-    bl_description = 'Parent selected objects to a new Empty'
-    bl_options = {'REGISTER', 'UNDO'}
 
-    nombre = StringProperty(name='', default='OBJECTS', description='Give the empty / group a name')
-    grupo = bpy.props.BoolProperty(name='Create Group', default=False, description='Also add objects to a group')
-    locat = bpy.props.EnumProperty(name='', items=[('CURSOR','Cursor','Cursor'),('ACTIVE','Active','Active'),
-    ('CENTER','Center','Selection Center')],description='Empty location', default='CENTER')
-    renom = bpy.props.BoolProperty(name='Add Prefix', default=False, description='Add prefix to objects name')
+def centro(sel):
+    x = sum([obj.location[0] for obj in sel]) / len(sel)
+    y = sum([obj.location[1] for obj in sel]) / len(sel)
+    z = sum([obj.location[2] for obj in sel]) / len(sel)
+    return (x, y, z)
+
+
+class P2E(Operator):
+    bl_idname = "object.parent_to_empty"
+    bl_label = "Parent to Empty"
+    bl_description = "Parent selected objects to a new Empty"
+    bl_options = {"REGISTER", "UNDO"}
+
+    nombre = StringProperty(
+                    name="",
+                    default='OBJECTS',
+                    description='Give the empty / group a name'
+                    )
+    grupo = BoolProperty(
+                    name="Create Group",
+                    default=False,
+                    description="Also add objects to a group"
+                    )
+    locat = EnumProperty(
+                    name='',
+                    items=[('CURSOR', 'Cursor', 'Cursor'), ('ACTIVE', 'Active', 'Active'),
+                           ('CENTER', 'Center', 'Selection Center')],
+                    description='Empty location',
+                    default='CENTER'
+                   )
+    renom = BoolProperty(
+                    name="Add Prefix",
+                    default=False,
+                    description="Add prefix to objects name"
+                    )
 
     @classmethod
     def poll(cls, context):
@@ -29,34 +52,38 @@ class P2E(bpy.types.Operator):
 
     def draw(self, context):
         layout = self.layout
-        layout.prop(self,'nombre')
+        layout.prop(self, "nombre")
         column = layout.column(align=True)
-        column.prop(self,'locat')
-        column.prop(self,'grupo')
-        column.prop(self,'renom')
+        column.prop(self, "locat")
+        column.prop(self, "grupo")
+        column.prop(self, "renom")
 
     def execute(self, context):
         objs = context.selected_objects
         act = context.object
         sce = context.scene
-        try: bpy.ops.object.mode_set()
-        except: pass
+
+        try:
+            bpy.ops.object.mode_set()
+        except:
+            pass
+
         if self.locat == 'CURSOR':
             loc = sce.cursor_location
         elif self.locat == 'ACTIVE':
             loc = act.location
         else:
-            loc = centro(objs) 
-        
-        bpy.ops.object.add(type='EMPTY',location=loc)
+            loc = centro(objs)
+
+        bpy.ops.object.add(type='EMPTY', location=loc)
         context.object.name = self.nombre
         context.object.show_name = True
         context.object.show_x_ray = True
-            
+
         if self.grupo:
             bpy.ops.group.create(name=self.nombre)
             bpy.ops.group.objects_add_active()
-            
+
         for o in objs:
             o.select = True
             if not o.parent:
@@ -66,50 +93,37 @@ class P2E(bpy.types.Operator):
             o.select = False
         for o in objs:
             if self.renom:
-                o.name = self.nombre+'_'+o.name
+                o.name = self.nombre + '_' + o.name
         return {'FINISHED'}
 
 
-class PreFix(bpy.types.Operator):
-    bl_idname = 'object.toggle_prefix'
-    bl_label = 'Toggle Sufix'
-    bl_description = 'Toggle parent name as sufix for c'
-    bl_options = {'REGISTER', 'UNDO'}
+class PreFix(Operator):
+    bl_idname = "object.toggle_prefix"
+    bl_label = "Toggle Sufix"
+    bl_description = "Toggle parent name as sufix for c"
+    bl_options = {"REGISTER", "UNDO"}
 
     @classmethod
     def poll(cls, context):
-        act = bpy.context.object
+        act = context.object
         return (act and act.type == 'EMPTY')
 
     def execute(self, context):
-        act = bpy.context.object
+        act = context.object
         objs = act.children
-        prefix = act.name+'_'
+        prefix = act.name + '_'
         remove = False
         for o in objs:
             if o.name.startswith(prefix):
                 remove = True
                 break
 
-        if remove == True:
+        if remove is True:
             for o in objs:
                 if o.name.startswith(prefix):
                     o.name = o.name.partition(prefix)[2]
         else:
             for o in objs:
-                o.name = prefix+o.name
+                o.name = prefix + o.name
 
         return {'FINISHED'}
-
-
-class PanelP2E(bpy.types.Panel):
-    bl_label = 'Parent to Empty'
-    bl_space_type = 'VIEW_3D'
-    bl_region_type = 'TOOLS'
-    bl_category = 'Relations'
-
-    def draw(self, context):
-        layout = self.layout
-        layout.operator('object.parent_to_empty')
-        layout.operator('object.toggle_prefix')
-