After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 603053 - "Import tags" category sounds like an action
"Import tags" category sounds like an action
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: Tags
0.6.x
Other Linux
: Normal normal
: ---
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks:
 
 
Reported: 2009-11-26 14:36 UTC by Vish
Modified: 2010-03-01 17:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Rename "Import Tags" to "Imported Tags" (1.65 KB, patch)
2010-01-06 18:16 UTC, Iain Churcher
none Details | Review
Rename "Import Tags" to "Imported Tags" Ver 2 (2.09 KB, patch)
2010-01-06 22:57 UTC, Iain Churcher
committed Details | Review

Description Vish 2009-11-26 14:36:20 UTC
Until recently I kept selecting the category "Import Tags" expecting the F-spot app to ask/allow me to import tags from some location/app.

The "Import tags" option sounds more like an action a user can perform rather than a collection of tags which are automatically added from the imported photos.

It would be better if this option is renamed as either "Imported Tags" or "External Tags" or something of the sort , so that the category sounds less like an action.
Comment 1 Iain Churcher 2010-01-06 18:16:14 UTC
Created attachment 150915 [details] [review]
Rename "Import Tags" to "Imported Tags"

Hi,

This bug has been marked as a lucid Paper cut,

As I am just starting out with some f-spot development I thought this would be an easy one to attempt.

The attached patch renames the "Import Tags" tag to "Imported Tags" to make it sound less like an action. 

However, name aside... (Please put forward a better name!) the patch only works for new users of f-spot as the "Import Tags" is actually stored in the photos.db in the tags table. Existing users who use the patch would find it would create another category "Imported Tags" and yet still display the old "Import Tags".

I could add code to check for the presence of the old tag and rename to the new one. But this seems like quite a hack.

How does f-spot handle database changes such as this? Is there a database update script or something that runs on install of a new f-spot version?

Regards

Iain Churcher
Comment 2 Iain Churcher 2010-01-06 22:57:47 UTC
Created attachment 150937 [details] [review]
Rename "Import Tags" to "Imported Tags" Ver 2

Hi,

Thanks to a tip from Maxxer I have updated the patch to update the Updater.cs file to create a new database update (17.1) which will rename the existing 'Import Tag' to 'Imported Tags'

This means for those with existing imported tags the patch will correctly work.

Comments (and any further suggestion to the name change) Welcome

Regards 

Iain Churcher
Comment 3 Ruben Vermeersch 2010-03-01 17:06:25 UTC
Looks good, merged. Thanks!