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.

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

wxGUI datacatalog: add direct editing of mapset and location name #920

Conversation

lindakarlovska
Copy link
Contributor

@lindakarlovska lindakarlovska commented Aug 20, 2020

Addresses both #917, #918.

#918:
Currently we can rename map by selecting it and clicking at it again, which opens an editing window (like in any file manager). This should work also for mapsets and locations. It should take into account if the restricted mode is on or off.

#917:
In datacatalog when we select a map in other mapset and click on it second time to open editing window, you can rename it even when restricted mode is on. This should not be permitted.

@petrasovaa petrasovaa added this to PRs in progress in New GUI Startup via automation Aug 20, 2020
@petrasovaa petrasovaa added gsoc Reserved for Google Summer of Code student(s) GUI wxGUI related labels Aug 20, 2020
gui/wxpython/startup/guiutils.py Outdated Show resolved Hide resolved
gui/wxpython/datacatalog/tree.py Show resolved Hide resolved
gui/wxpython/datacatalog/tree.py Outdated Show resolved Hide resolved
gui/wxpython/datacatalog/tree.py Outdated Show resolved Hide resolved
@lindakarlovska lindakarlovska self-assigned this Aug 21, 2020
gui/wxpython/datacatalog/tree.py Outdated Show resolved Hide resolved
gui/wxpython/datacatalog/tree.py Show resolved Hide resolved
gui/wxpython/datacatalog/tree.py Outdated Show resolved Hide resolved
gui/wxpython/datacatalog/tree.py Outdated Show resolved Hide resolved
gui/wxpython/startup/guiutils.py Outdated Show resolved Hide resolved
lib/python/grassdb/checks.py Outdated Show resolved Hide resolved
gui/wxpython/startup/guiutils.py Outdated Show resolved Hide resolved
gui/wxpython/startup/guiutils.py Outdated Show resolved Hide resolved
Copy link
Contributor

@petrasovaa petrasovaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks ok, still GetListOfLocations should be moved as you suggested, see my comment. Another change are the conditions in OnStartEditLabel, I suggested some changes there. This PR has conflicts, please resolve them.

gui/wxpython/datacatalog/tree.py Outdated Show resolved Hide resolved
lindakladivova and others added 5 commits August 23, 2020 03:34
…SGeo#918. Created two check functions - for new mapset name and new location name, results of those functions are displayed as Notification.
…from guiutils.py), also existing functions as get_reason_mapset_removable moved to checks.py. get_reason_grassdb_removable was not moved because of GetListOfLocations function. MapsetDialog and LocationDialog a bit changed, now it uses GenericValidator instread of MultiGenericValidator.
Co-authored-by: Anna Petrasova <kratochanna@gmail.com>
@lindakarlovska lindakarlovska force-pushed the wxGUI-add-direct-editing-of-mapset-and-location-name branch from 77da00e to ead598f Compare August 23, 2020 08:36
…ditions in OnStartEditLabel made more straightforward.
@petrasovaa petrasovaa merged commit 80538be into OSGeo:master Aug 24, 2020
New GUI Startup automation moved this from PRs in progress to Done Aug 24, 2020
@neteler neteler added this to the 8.0.0 milestone Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gsoc Reserved for Google Summer of Code student(s) GUI wxGUI related
Projects
No open projects
4 participants