bug fix: 2628615, 2625434, 2625090

CHANGELOG.txt

@@ -5,6 +5,18 @@ Started 2007-June-11
 Please add newer entries at the top, list the date and your name with
 email address.
 
+2009-Feb-24 UPDATE Jean-Pierre Charras <jean-pierre.charras@inpg.fr>
+================================================================================
+++pcbnew:
+    bug fix:
+        2628615 	  Error message after right click into zone
+        2625434  	  Crash after trying to edit a line/drawing
+    
+++cvpcb
+    fix: 2625090  CVpcb - language selection
+    
+    
+
 2009-Feb-17 UPDATE Jean-Pierre Charras <jean-pierre.charras@inpg.fr>
 ================================================================================
 ++gr_basic.cpp

cvpcb/cvframe.cpp

@@ -488,6 +488,8 @@ void WinEDA_CvpcbFrame::SetLanguage( wxCommandEvent& event )
 
     wxGetApp().SetLanguageIdentifier( id );
     wxGetApp().SetLanguage();
+    ReCreateMenuBar();
+    Refresh();
 }
 
 

include/build_version.h

@@ -3,13 +3,15 @@
 #ifndef KICAD_BUILD_VERSION
 #define KICAD_BUILD_VERSION
 
+#define BUILD_VERSION wxT("(20090216-RC3)")
+
 COMMON_GLOBL wxString g_BuildVersion
 #ifdef EDA_BASE
 #  ifdef HAVE_SVN_VERSION
 #    include "config.h"
      (wxT(KICAD_SVN_VERSION))
 #  else
-     (wxT("(20090216-RC2)")) /* main program version */
+     (BUILD_VERSION) /* main program version */
 #  endif
 #endif
 ;
@@ -20,7 +22,7 @@ COMMON_GLOBL wxString g_BuildAboutVersion
 #    include "config.h"
      (wxT(KICAD_ABOUT_VERSION))
 #  else
-     (wxT("(20090216-RC2)")) /* svn date & rev (normally overridden) */
+     (BUILD_VERSION) /* svn date & rev (normally overridden) */
 #  endif
 #endif
 ;

pcbnew/dialog_graphic_item_properties.cpp

@@ -13,6 +13,7 @@
 #include "fctsys.h"
 #include "macros.h"
 #include "common.h"
+#include "confirm.h"
 #include "class_drawpanel.h"
 #include "pcbnew.h"
 
@@ -51,6 +52,11 @@ DialogGraphicItemProperties::DialogGraphicItemProperties( WinEDA_BasePcbFrame* a
 void WinEDA_BasePcbFrame::InstallGraphicItemPropertiesDialog(DRAWSEGMENT * aItem, wxDC* aDC)
 /*******************************************************************************************/
 {
+    if ( aItem == NULL )
+    {
+        DisplayError(this, wxT("nstallGraphicItemPropertiesDialog() error: NULL item"));
+        return;
+    }
     DrawPanel->m_IgnoreMouseEvents = TRUE;
     DialogGraphicItemProperties* dialog = new DialogGraphicItemProperties( this,
         aItem, aDC );

pcbnew/edit.cpp

@@ -135,6 +135,7 @@ void WinEDA_PcbFrame::Process_Special_Functions( wxCommandEvent& event )
     case ID_POPUP_PCB_VIA_HOLE_EXPORT:
     case ID_POPUP_PCB_VIA_HOLE_RESET_TO_DEFAULT:
     case ID_POPUP_PCB_VIA_HOLE_EXPORT_TO_OTHERS:
+    case ID_POPUP_PCB_EDIT_DRAWING:
         break;
 
     case ID_POPUP_CANCEL_CURRENT_COMMAND:
@@ -574,13 +575,13 @@ void WinEDA_PcbFrame::Process_Special_Functions( wxCommandEvent& event )
         break;
 
     case ID_POPUP_PCB_REMOVE_FILLED_AREAS_IN_ALL_ZONES: // Remove all zones :
-        GetBoard()->m_Zone.DeleteAll();
+        GetBoard()->m_Zone.DeleteAll();    // remove zone segments used to fill zones.
         for( int ii = 0; ii < GetBoard()->GetAreaCount(); ii++ )
-        {
+        {   // Remove filled aresa in zone
             ZONE_CONTAINER* zone_container = GetBoard()->GetArea( ii );
             zone_container->m_FilledPolysList.clear();;
         }
-
+        SetCurItem(NULL);           // CurItem might be deleted by this command, clear the pointer
         test_connexions( NULL );
         Tst_Ratsnest( NULL, 0 );    // Recalculate the active ratsnest, i.e. the unconnected links */
         GetScreen()->SetModify();

pcbnew/zones_by_polygon.cpp

@@ -127,8 +127,11 @@ void WinEDA_PcbFrame::Delete_Zone_Fill( wxDC* DC, SEGZONE* aZone, long aTimestam
     {
         ZONE_CONTAINER* zone = GetBoard()->GetArea( ii );
         if( zone->m_TimeStamp == TimeStamp )
+        {
+            modify = TRUE;
             zone->m_FilledPolysList.clear();
         }
+    }
 
     if( modify )
     {