Bug report #21254

Layer Properties > Load Style > From database > 'Other styles ...' > Description column contains Name text, should contain Description text (it is a geopackage layer)

Added by Steve Lowman over 5 years ago. Updated over 5 years ago.

Status:Closed
Priority:Normal
Assignee:Alessandro Pasotti
Category:Symbology
Affected QGIS version:3.5(master) Regression?:No
Operating System:Any Easy fix?:No
Pull Request or Patch supplied:Yes Resolution:fixed/implemented
Crashes QGIS or corrupts data:No Copied to github as #:29072

Description

Using Layer Properties > Style > Save Style, I save a style for a geopackage layer, and I enter the 'Name' and a slightly longer text for the 'Description'. Then, I go into the Load Style dialog for a layer in the same geopackage.

In the 'Styles related to the layer' table list, the 'Name' text and the 'Description' text are both correctly displaying what I entered.

However, in the 'Other styles on the database' table list, the 'Description' column is showing the same text as in the 'Name' column. It should display the text that I entered for 'Description' when I saved the styles.

(I do not know what category to make this report - maybe 'Forms'?)

Layer Properties - Save Styles - Dbase Styles Manager - Other styles - Description issue.png - Image showing the dialog with the incorrect text in the 'Description' column of 'Other styles on the database' table. (49 KB) Steve Lowman, 2019-02-13 12:44 PM

Associated revisions

Revision 3e041b3a
Added by Alessandro Pasotti over 5 years ago

[ogr] Show style description instead of name in other styles

Fixes #21254

History

#1 Updated by Alessandro Pasotti over 5 years ago

  • Assignee set to Alessandro Pasotti

While I'm on styles I can have a look

#2 Updated by Giovanni Manghi over 5 years ago

  • Category changed from Unknown to Symbology

#3 Updated by Steve Lowman over 5 years ago

A slightly off-topic suggestion:

Perhaps for the same dialog, it could be worth considering changing the default value in the the drop-down list at the top, when it is a geopackage layer, to be 'From database' (for the Load Style dialog) or 'In database' (for the Save Style dialog), instead of defaulting to the .qml file option.

#4 Updated by Alessandro Pasotti over 5 years ago

Actually, I've been thinking for a while at remembering last user's choice (and I've just finished to implement it).

#5 Updated by Alessandro Pasotti over 5 years ago

  • Status changed from Open to In Progress
  • Operating System changed from Windows 10 to Any
  • Resolution set to fixed/implemented
  • Pull Request or Patch supplied changed from No to Yes
  • Affected QGIS version changed from 3.4.4 to 3.5(master)

#6 Updated by Steve Lowman over 5 years ago

That's great, very quick fix. Thank you for the fine work. I notice you changed the affected version to 3.5. Will the fix apply to 3.4.5 LTR?

#7 Updated by Alessandro Pasotti over 5 years ago

Np, thanks for reporting.

Yes: this will be backported to 3.4 release branch

#8 Updated by Alessandro Pasotti over 5 years ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Closed

Also available in: Atom PDF