Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

wxGUI: Fixed E722 in frame.py #4440

Merged
merged 7 commits into from
Oct 10, 2024
Merged

wxGUI: Fixed E722 in frame.py #4440

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
9404534
fixed e722
Oct 3, 2024
1f5d777
updated acc. to comments
Oct 8, 2024
0a8837e
fixed e266
Oct 8, 2024
2ee5130
updated flake8
Oct 9, 2024
0836087
updated
Oct 9, 2024
eb39b08
Merge branch 'main' into 722-frame
arohanajit Oct 9, 2024
19b7bdd
updated
Oct 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions gui/wxpython/lmgr/frame.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -271,7 +271,7 @@ def show_menu_errors(messages):
y = client_disp[1]
self.SetPosition((x, y))
self.SetSize((w, h))
except:
except (ValueError, IndexError):
pass
else:
# does center (of screen) make sense for lmgr?
Expand Down Expand Up @@ -885,7 +885,7 @@ def OnCBPageChanged(self, event):
try:
self.GetMapDisplay().SetFocus()
self.GetMapDisplay().Raise()
except:
except Exception:
arohanajit marked this conversation as resolved.
Show resolved Hide resolved
pass

event.Skip()
Expand Down Expand Up @@ -1125,7 +1125,7 @@ def GetMenuCmd(self, event):

try:
cmdlist = cmd.split(" ")
except: # already list?
except AttributeError: # already list?
cmdlist = cmd

# check list of dummy commands for GUI modules that do not have GRASS
Expand All @@ -1137,7 +1137,7 @@ def GetMenuCmd(self, event):
layer = self.GetLayerTree().layer_selected
name = self.GetLayerTree().GetLayerInfo(layer, key="maplayer").name
type = self.GetLayerTree().GetLayerInfo(layer, key="type")
except:
except (AttributeError, KeyError):
arohanajit marked this conversation as resolved.
Show resolved Hide resolved
layer = None

if layer and len(cmdlist) == 1: # only if no parameters given
Expand Down Expand Up @@ -1185,7 +1185,7 @@ def OnVDigit(self, event):
# available only for vector map layers
try:
mapLayer = tree.GetLayerInfo(layer, key="maplayer")
except:
except (AttributeError, KeyError):
mapLayer = None

if not mapLayer or mapLayer.GetType() != "vector":
Expand Down Expand Up @@ -1862,7 +1862,7 @@ def OnShowAttributeTable(self, event, selection=None):
# available only for vector map layers
try:
maptype = tree.GetLayerInfo(layer, key="maplayer").type
except:
except (AttributeError, KeyError):
arohanajit marked this conversation as resolved.
Show resolved Hide resolved
maptype = None

if not maptype or maptype != "vector":
Expand Down
Loading