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

Attachment widget options are reset when you reopen the field widget properties #45089

Closed
1 of 2 tasks
DelazJ opened this issue Sep 16, 2021 · 6 comments · Fixed by #45090
Closed
1 of 2 tasks

Attachment widget options are reset when you reopen the field widget properties #45089

DelazJ opened this issue Sep 16, 2021 · 6 comments · Fixed by #45090
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Widgets

Comments

@DelazJ
Copy link
Contributor

DelazJ commented Sep 16, 2021

What is the bug or the crash?

Both "storage mode" and "Relative paths" buttons are reset to default when you reopen the field widget properties
attachmentcheckboxes

Steps to reproduce the issue

See above

Versions

master 3.21

Supported QGIS version

  • I'm running a supported QGIS version according to the roadmap.

New profile

  • I tried with a new QGIS profile

Additional context

I think it's due to some widget visibility checks in b1fa333

@DelazJ DelazJ added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label Sep 16, 2021
@gioman gioman added the Widgets label Sep 16, 2021
@gioman
Copy link
Contributor

gioman commented Sep 16, 2021

@DelazJ this is a regression, right?

@DelazJ
Copy link
Contributor Author

DelazJ commented Sep 16, 2021

No. I don't think so though i did not test latest 3.16. New features added to master led to the changes and no reason to backport.

@troopa81
Copy link
Contributor

It's a duplicate of #44799 and I proposed a PR #45072

@DelazJ
Copy link
Contributor Author

DelazJ commented Sep 16, 2021

Ah sorry. I missed that ones.
While you are here, is the "path" group necessary when the storage type is not "existing file"?

@troopa81
Copy link
Contributor

While you are here, is the "path" group necessary when the storage type is not "existing file"?

Yes, it's used as the default folder opened when you select a file from the file selector widget.

@DelazJ
Copy link
Contributor Author

DelazJ commented Sep 16, 2021

Thanks @troopa81

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Widgets
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants