Ps. Just tried adding a row to cmsTags, and one to cmsTagRelationship. This behaves just like the ones created programmatically (i.e. not showing up in the property on the image).

Hi Kenneth,

Not sure if you just haven't pasted the code but are you saving the IContent after calling SetTags?

Thanks,

Tom

Hi Tom,

Yes, I have tried various combinations of:

Create image, save, set tags

Create image, set tags, save

Create image, save, get image to new variable, set tags, save.

This is the code as it is presently (not working):

var buffer = await file.ReadAsByteArrayAsync();
                var stream = new MemoryStream(buffer);
                var mediaService = Services.MediaService;
                var image = mediaService.CreateMedia(filename, parentId, "Image");
                image.SetValue("umbracoFile", filename, stream);
                mediaService.Save(image);

                // Tags
                var imageForTags = mediaService.GetById(image.Id);
                List<string> separators = new List<string>() { "¤¤" };
                var tags = tagString.Split(separators.ToArray(), StringSplitOptions.RemoveEmptyEntries).Take(3);
                imageForTags.SetTags("tags", tags, true);
                mediaService.Save(imageForTags);

                id = image.Id;

BR. Kenneth.

Do you get any errors in the browser console when viewing the page in Umbraco?

Thanks,

Tom

Aha! Yes. I get the following:

BR. Kenneth.

The things I am trying to tag on are things like product number, brand, type etc., given as parts of the filename.

I loop through a lot of images, so some of them are bound to have the same brand, f.ex.

I'm not sure if the same term exists more than once in the filename, but maybe I should do a distinct on my "tags" variable?

Hi Kenneth,

Maybe trying calling .Distinct() on the tags array when passing it to SetTags. That error message makes it sound as if there are duplicate tags.

Thanks,

Tom

Calling Distinct() made no difference.

I then tried a simple experiment, just retrieving an existing image, setting tags, and saving. That doesn't seem to work either.

I still get the "Error: Duplicates in a repeater are not allowed. Repeater: tag in model.value" in the console when viewing the image in the backoffice.

The code has now been boiled down to:

        var service = Services.MediaService;
        var item = service.GetById(2120);
        var tags = new List<string>() { "Test1", "Test2", "Test3" };
        item.SetTags("tags", tags, true);
        service.Save(item);

I still get the rows in cmsTags and cmsTagRelationship, and everything seems to look OK in the DB.

I'm thinking the next step is to compare the entire DB to see how it looks when 1) Tags are entered manually. 2) Tag are created with SetTags.

There must be something that SetTags doesn't do, which is needed for the tags to show correctly.

BR. Kenneth.

BR. Kenneth.

OK, so I did the following:

1) Start with no tags on any image.

2) Add a tag to an image.

3) Make a DB backup.

4) Add a second tag to the image.

5) Write down which tables had new/updated data

6) Make a DB backup.

7) Add a third tag programmatically using .SetTags().

8) Compare DB after third tag was added.

So, all the same tables were touched, which were the following:

• Contentversion - update versiondate field

• Contentxml - update xml field

• Cmspropertydata - update datanvarchar field

• Cmstag - insert 1 row

• Cmstagrelationship - insert 1 row

• Cmscacheinstruction - insert 1 row

• Umbracolog - insert 1 row

• Umbraconode - update 2 rows

• Umbracoserver - update 2 rows

However, the one called cmsPropertyData looks different after adding the third tag using SetTags.

Looks like the data is no longer represented as a JSON array, but just a comma-separated list of the tags. Could this be a bug?

I noticed in the Tags data type, that I can choose between "JSON" and "CSV in "Storage Type".

Mine is set to "JSON".

Could it be that SetTags does not take this into account, and just always store it as CSV, whereas manually entering tags will honor "Storage Type"?

BR. Kenneth.

No worries, glad you have it all working now :)

I know this is a little old, but I was looking at the code, shouldn't the storage group "default" use what is set in the control, instead of default? Am I missing something?

is working on a reply...