Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #10393 (closed)

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

Bug: Tags on import with ns:""

Reported by: khgillen Owned by: jburel
Priority: major Milestone: OMERO-4.4.7
Component: Insight Version: n.a.
Keywords: n.a. Cc: ux@…, jamoore, wmoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: Bugs Fixing

Description (last modified by khgillen)

Using FS testing release Gretzky, also seen on 4.4.6 release.

Batch adding tags during/post import with the importer "add tags to images" option will tag images, but when viewing the 'tags' view in Web, the tags created by the importer don't show up.

They do appear in the annotations window in web, but when viewing by tag (https://gretzky.openmicroscopy.org.uk/omero/webclient/usertags/) they don't show up.

Adding tags from Insight, the tags show up in Web, just not when created with the importer.

Also - not sure if this is the same, or a different bug, but the thumbnails in Insight don't have the annotation symbol when they have tags assigned only by the importer - once you assign the tag within insight, and they become visible to Web, they also acquire their thumbnail with the annotation symbol.

Change History (13)

comment:1 Changed 12 years ago by khgillen

  • Description modified (diff)

comment:2 Changed 12 years ago by khgillen

  • Milestone changed from Unscheduled to OMERO-4.5

comment:3 Changed 12 years ago by wmoore

Tags created in the Insight importer have an empty string as namespace (if created elsewhere, ns is Null). The empty string ns prevents them appearing in the web as we get orphaned tags with:

sql = "select tg from TagAnnotation tg where ((ns=:ns) or (ns is null and not exists ( select aal from AnnotationAnnotationLink as aal where aal.child=tg.id))) "

comment:4 Changed 12 years ago by wmoore

This has also been reported in #10248 and #10405

comment:5 Changed 12 years ago by wmoore

  • Resolution set to fixed
  • Status changed from new to closed

comment:6 Changed 12 years ago by wmoore

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Summary changed from Bug: Batch Added Tags at Import don't show up in Tags view in Web to Bug: Tags on import with ns:""

Reassigning to JM to fix the issue of Insight writing empty strings as the namespace for tags created at import.

Re: Kenny's comment:

Also - not sure if this is the same, or a different bug, but the thumbnails in Insight don't have the annotation symbol when they have tags assigned only by the importer - once you assign the tag within insight, and they become visible to Web, they also acquire their thumbnail with the annotation symbol.

comment:7 Changed 12 years ago by wmoore

  • Component changed from Web to Insight
  • Owner changed from wmoore to jburel
  • Status changed from reopened to accepted

comment:8 Changed 12 years ago by jburel

The icon is due to the count of annotation not working correctly server side. We have a ticket for that.

comment:9 Changed 12 years ago by jburel

  • Sprint set to Bugs Fixing

comment:10 Changed 12 years ago by jburel

  • Cc jmoore wmoore added

All the annotations created using Annotations Pojos objects have been created using an empty string in the namespace i.e.

    protected AnnotationData(Class<? extends Annotation> annotationClass) {
        if (annotationClass == null) {
            throw new IllegalArgumentException("annotationClass cannot be null");
        }
        try {
            Annotation a = annotationClass.newInstance();
            a.setNs(rstring(""));
            setValue(a);
        } catch (Exception e) {
            throw new IllegalArgumentException("Unkown annotation type: "
                    + annotationClass.getName());
        }
    }

This will probably require an upgrade (for develop) to replace the empty string by null.

comment:11 Changed 12 years ago by jburel

  • Resolution set to fixed
  • Status changed from accepted to closed

comment:12 Changed 12 years ago by jburel <j.burel@…>

(In [08c2d8b1378d21143d13830336a877f6b61d884a/ome.git] on branch develop) Do not set the namespace to an empty string (see #10393)

comment:13 Changed 12 years ago by Will Moore <will@…>

(In [bd5be6dfd6cdc5d861237ff4dd9836e9b8ce94ef/ome.git] on branch develop) Web listTags() includes Tags with empty string Namespace. See #10393

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.64787 sec.)

We're Hiring!