wxGUI: display attributes bug fix (not found --- found)

(merge devbr6, https://trac.osgeo.org/grass/changeset/33283, https://trac.osgeo.org/grass/changeset/33284)


git-svn-id: https://svn.osgeo.org/grass/grass/trunk@33285 15284696-431f-4ddb-bdfa-cd5b030d7da7

gui/wxpython/gui_modules/dbm.py

@@ -2805,11 +2805,15 @@ class DisplayAttributesDialog(wx.Dialog):
         mainSizer.Fit(self)
 
         # set min size for dialog
-        self.SetMinSize(self.GetBestSize())
+        w, h = self.GetBestSize()
+        if h < 200:
+            self.SetMinSize((w, 200))
+        else:
+            self.SetMinSize(self.GetBestSize())
 
         if self.notebook.GetPageCount() == 0:
             Debug.msg(2, "DisplayAttributesDialog(): Nothing found!")
-            self.mapDBInfo = None
+            ### self.mapDBInfo = None
 
     def __SelectAttributes(self, layer):
         """Select attributes"""
@@ -2958,7 +2962,7 @@ class DisplayAttributesDialog(wx.Dialog):
             self.mapDBInfo = VectorDBInfo(self.map)
             
         self.line['id'] = (line, )
-        self.line['cats'] = None
+        self.line['cats'] = cats
         
         if not self.mapDBInfo:
             return False
@@ -3103,9 +3107,16 @@ class DisplayAttributesDialog(wx.Dialog):
         # for each layer END
 
         self.Layout()
-
+        
         return True
 
+    def IsFound(self):
+        """Check if attributes found"""
+        if self.notebook.GetPageCount() > 0:
+            return True
+        
+        return False
+        
 class VectorDBInfo(gselect.VectorDBInfo):
     """Class providing information about attribute tables
     linked to the vector map"""

gui/wxpython/gui_modules/mapdisp.py

@@ -1223,9 +1223,9 @@ class BufferedWindow(MapWindow, wx.Window):
                                                                            line=line)
 
                     if self.parent.dialogs['attributes']:
-                        line = self.parent.dialogs['attributes'].GetLine()
-                        if self.parent.dialogs['attributes'].mapDBInfo and line:
+                        if self.parent.dialogs['attributes'].IsFound():
                             # highlight feature & re-draw map
+                            line = self.parent.dialogs['attributes'].GetLine()
                             digitClass.driver.SetSelected(line)
                             if not self.parent.dialogs['attributes'].IsShown():
                                 self.parent.dialogs['attributes'].Show()

gui/wxpython/gui_modules/vdigit.py

@@ -1151,7 +1151,7 @@ class VDigit(AbstractDigit):
         
         @param line feature id (-1 for first selected line)
         """
-        return self.digit.GetLineCats(line)
+        return dict(self.digit.GetLineCats(line))
 
     def SetLineCats(self, line, layer, cats, add=True):
         """Set categories for given line and layer

gui/wxpython/vdigit/cats.cpp

@@ -158,9 +158,8 @@ std::vector<int> Digit::GetLayers()
 std::map<int, std::vector<int> > Digit::GetLineCats(int line_id)
 {
     std::map<int, std::vector<int> > lc;
-    int line, n_dblinks;
+    int line;
     struct line_cats *Cats;
-    struct field_info *fi;
 
     if (!display->mapInfo) {
 	DisplayMsg();
@@ -189,20 +188,12 @@ std::map<int, std::vector<int> > Digit::GetLineCats(int line_id)
 	ReadLineMsg(line);
 	return lc;
     }
-
-    n_dblinks = Vect_get_num_dblinks(display->mapInfo);
-
-    for (int dblink = 0; dblink < n_dblinks; dblink++) {
-	fi = Vect_get_dblink(display->mapInfo, dblink);
-	if (fi == NULL) {
-	    DblinkMsg(dblink+1);
-	    continue;
-	}
-	std::vector<int> cats;
-	lc[fi->number] = cats;
-    }
-
+    
     for (int i = 0; i < Cats->n_cats; i++) {
+	if (lc.find(Cats->field[i]) == lc.end()) {
+	    std::vector<int> cats;
+	    lc[Cats->field[i]] = cats;
+	}
 	lc[Cats->field[i]].push_back(Cats->cat[i]);
     }