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

catchment area not selected with "Select current path" #566

Closed
urskaufmann opened this issue Mar 23, 2025 · 3 comments
Closed

catchment area not selected with "Select current path" #566

urskaufmann opened this issue Mar 23, 2025 · 3 comments
Labels
bug Something isn't working plugin Concerns the wastewater plugin

Comments

@urskaufmann
Copy link
Contributor

Describe the bug
one of the missing feature is, that the user has no tool to select the catchment areas, that where connected to selected nodes. That's why I search for code in the current plugin. And I found in \python\plugins\teksi_wastewater\gui\twwprofiledockwidget.py some code, that looks like tww should select not only nodes, but also catchment areas when choosing the Select current path command. Line 173ff

But in line 173 is code that can not work, because it is from qgep.
catchment_areas_layer = TwwLayerManager.layer("od_catchment_area")
In qgep we had layers with od_*. !

It works, if I change this line to
catchment_areas_layer = TwwLayerManager.layer("catchment_area")

To Reproduce
Exact steps to reproduce the behavior:

  • use the upstream network following tool
  • click on "Select current path"
  • no catchment area are selected

Desktop (please complete the following information):

  • TWW version 2024.0.5
  • QGIS Version 3.40.4
  • OS Win 10

Little change to use the code, that already exists and to help me on my daily work.

The help would be bigger, if the user can select the nodes and catchment_areas starting with a selection of reaches (separat tool)

@cymed cymed added bug Something isn't working plugin Concerns the wastewater plugin labels Mar 24, 2025
@cymed
Copy link
Contributor

cymed commented Mar 24, 2025

Thanks for pointing that out! Could you provide a pull request?

@ponceta
Copy link
Member

ponceta commented Mar 26, 2025

For the future, we should ensure that layer names syntax or translations are not affecting actions or plugin.

@cymed
Copy link
Contributor

cymed commented Mar 26, 2025

Solved with #567

@cymed cymed closed this as completed Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working plugin Concerns the wastewater plugin
Projects
None yet
Development

No branches or pull requests

3 participants