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

keep provider selection on re-open #169

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

keep provider selection on re-open #169

wants to merge 3 commits into from

Commits on Mar 9, 2022

  1. Added provider storage for ORStoolsDialog 

    the provider choice is now persisted in the config.yml file
and automatically selected in the ORStoolsDialog window.

The configmanager was extended and can now set and get the active provider
    @TheGreatRefrigerator
    TheGreatRefrigerator committed Mar 9, 2022
    Configuration menu
    Copy the full SHA
    876b2bd View commit details
    Browse the repository at this point in the history

  2. Move ors_client to base_processing_algorithm 

    small refactor moving the ors_client and provider logic
to the base processing algorithm class.

processAlgorithm methods now use the complete signature
    @TheGreatRefrigerator
    TheGreatRefrigerator committed Mar 9, 2022
    Configuration menu
    Copy the full SHA
    ffbc8dc View commit details
    Browse the repository at this point in the history

  3. Get and set active provider in processing algorithms 

    to use the active provider in processing algorithms by default,
it is read from the storage.
However, the defaultValue for QgsProcessingParameterEnum is not working,
which is why the options of the enum are reordered to show the active
provider on top.
As the index of the provider selected in the Enum does not refer to the
same provider in the config anymore, it has to be resolved properly again,
when using the provider to generate the ors_client.

This can lead to problems (using the wrong provider) when the active provider
is changed, after a processing algorithm window was opened, due to how
the algorithm parameters are initialized.
    @TheGreatRefrigerator
    TheGreatRefrigerator committed Mar 9, 2022
    Configuration menu
    Copy the full SHA
    81d65de View commit details
    Browse the repository at this point in the history