pcbnew: Add a number of missing handles for module zones

pcbnew/board_commit.cpp

@@ -149,7 +149,8 @@ void BOARD_COMMIT::Push( const wxString& aMessage, bool aCreateUndoEntry, bool a
                         board->Modules().front()->Add( boardItem );
                 }
                 else if( boardItem->Type() == PCB_MODULE_TEXT_T ||
-                         boardItem->Type() == PCB_MODULE_EDGE_T )
+                         boardItem->Type() == PCB_MODULE_EDGE_T ||
+                         boardItem->Type() == PCB_MODULE_ZONE_AREA_T )
                 {
                     wxASSERT( boardItem->GetParent() &&
                               boardItem->GetParent()->Type() == PCB_MODULE_T );

pcbnew/class_board.cpp

@@ -941,6 +941,7 @@ SEARCH_RESULT BOARD::Visit( INSPECTOR inspector, void* testData, const KICAD_T s
         case PCB_PAD_T:
         case PCB_MODULE_TEXT_T:
         case PCB_MODULE_EDGE_T:
+        case PCB_MODULE_ZONE_AREA_T:
 
             // this calls MODULE::Visit() on each module.
             result = IterateForward<MODULE*>( m_modules, inspector, testData, p );
@@ -954,6 +955,7 @@ SEARCH_RESULT BOARD::Visit( INSPECTOR inspector, void* testData, const KICAD_T s
                 case PCB_PAD_T:
                 case PCB_MODULE_TEXT_T:
                 case PCB_MODULE_EDGE_T:
+                case PCB_MODULE_ZONE_AREA_T:
                     continue;
 
                 default:

pcbnew/kicad_clipboard.cpp

@@ -82,6 +82,7 @@ void CLIPBOARD_IO::SaveSelection( const PCBNEW_SELECTION& aSelected )
         // check if it not one of the module primitives
         if( ( i->Type() != PCB_MODULE_EDGE_T ) &&
                 ( i->Type() != PCB_MODULE_TEXT_T ) &&
+                ( i->Type() != PCB_MODULE_ZONE_AREA_T ) &&
                 ( i->Type() != PCB_PAD_T ) )
         {
             onlyModuleParts = false;
@@ -167,6 +168,7 @@ void CLIPBOARD_IO::SaveSelection( const PCBNEW_SELECTION& aSelected )
             // Dont format stuff that cannot exist standalone!
             if( ( i->Type() != PCB_MODULE_EDGE_T ) &&
                 ( i->Type() != PCB_MODULE_TEXT_T ) &&
+                ( i->Type() != PCB_MODULE_ZONE_AREA_T ) &&
                 ( i->Type() != PCB_PAD_T ) )
             {
                 auto item = static_cast<BOARD_ITEM*>( i );

pcbnew/tools/edit_tool.cpp

@@ -762,6 +762,7 @@ int EDIT_TOOL::Mirror( const TOOL_EVENT& aEvent )
         {
         case PCB_MODULE_EDGE_T:
         case PCB_MODULE_TEXT_T:
+        case PCB_MODULE_ZONE_AREA_T:
         case PCB_PAD_T:
             // Only create undo entry for items on the board
             if( !item->IsNew() && !EditingModules() )
@@ -782,6 +783,13 @@ int EDIT_TOOL::Mirror( const TOOL_EVENT& aEvent )
             break;
         }
 
+        case PCB_MODULE_ZONE_AREA_T:
+        {
+            auto& zone = static_cast<MODULE_ZONE_CONTAINER&>( *item );
+            zone.Mirror( mirrorPoint, false );
+            break;
+        }
+
         case PCB_MODULE_TEXT_T:
         {
             auto& modText = static_cast<TEXTE_MODULE&>( *item );

pcbnew/tools/selection_tool.cpp

@@ -646,6 +646,7 @@ SELECTION_LOCK_FLAGS SELECTION_TOOL::CheckLock()
 
         case PCB_MODULE_EDGE_T:
         case PCB_MODULE_TEXT_T:
+        case PCB_MODULE_ZONE_AREA_T:
             if( static_cast<MODULE*>( item->GetParent() )->IsLocked() )
                 containsLocked = true;
             break;
@@ -1546,6 +1547,7 @@ bool SELECTION_TOOL::Selectable( const BOARD_ITEM* aItem, bool checkVisibilityOn
     switch( aItem->Type() )
     {
     case PCB_ZONE_AREA_T:
+    case PCB_MODULE_ZONE_AREA_T:
     {
         // Check to see if this keepout is part of a footprint
         // If it is, and we are not editing the footprint, it should not be selectable