Skip to content
This repository has been archived by the owner on Aug 10, 2024. It is now read-only.

List-type tags such as Keywords return concatenated string of all values. #9

Closed
dm413 opened this issue Dec 13, 2018 · 7 comments
Closed

Comments

@dm413
Copy link

dm413 commented Dec 13, 2018

I am trying to change the IPTC keywords tag. The GetTagsAsync method returns a tag with name "Keywords", but the value is a single string with all the keywords concatenated without separators. I would prefer a List<string>, but at the very least we need quote marks and separators.

Then when writing the keywords using OverwriteTagsAsync, how would I create the SetOperation?

Thanks

@AerisG222
Copy link
Owner

Thanks for the report. I have a lot going on at the moment, but will try to find some time to have a look at this. If possible, could you attach a sample image with those fields populated? feel free to black out the actual image

Thanks

@dm413
Copy link
Author

dm413 commented Dec 14, 2018

Attached is HalfMoonOverhang.jpg. I used Irfanview to add a title, description, and 3 keywords:

    Overhang
    Fay Canyon
    RedRocks

I deliberately included a keyword with a space because people do use them.
halfmoonoverhang

@AerisG222
Copy link
Owner

This is great - thank you for the sample!

@AerisG222
Copy link
Owner

@dm413 I've just published a v0.9.0-beta which hopefully will appear as an update if you enable pre-release packages. I made a lot of changes that I think will be more maintainable, but does introduce a lot of breaking changes, but should be pretty straightforward to address.

As an example for your read and write scenarios, please look at ParseTests.cs:GetTagsFromImageWithKeywords() and any of the tests in WriteTests.cs. When you have some time, give it a try and let me know if it works for you, and if you have any suggestions on how to improve.

Thanks!

@AerisG222
Copy link
Owner

Sorry, to view the examples in the test code, you will need to look in the overhaul branch

@dm413
Copy link
Author

dm413 commented Dec 30, 2018

Sorry for the delay. I tested the pre-release version and it works for me. Thanks!

@AerisG222
Copy link
Owner

Thanks for testing this out! v0.9.0 should be available on nuget now, with no changes other than version number.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants