- edited description
URL for related tags relevant for group XY
While viewing items relevant for a group, it is not possible to show items in that group that are tagged with a related tag (see sidebar). The URL is created falsely: it should be /relevantfor/group/GROUP_NAME/RELATED_TAG
instead of relevantfor/group/RELATED_TAG
Comments (19)
-
reporter -
reporter - edited description
-
-
assigned issue to
-
assigned issue to
-
-
assigned issue to
-
assigned issue to
-
- changed status to open
-
- changed version to 2.0.43
-
- changed version to 2.0.44
-
- changed version to 2.0.45
-
- changed version to 2.0.46
-
the ${requPath} value contains the needed information 'relevantfor/group/GROUPNAME' but the problem here is the function 'getLowerPath()' [Functions.java] being called in related.tagx when the single-tag-url is constructed.
/** * Cuts the last segment of the url string until last slash. TODO: If the * path contains more than three slashes, then the cut is after the third * slash. (Previous to adding this restrictions, tags that included a slash * could not be handled on /user/USER/TAG; remove as soon as the bug in * urlrewrite lib is fixed * * @param uriString * the url * @return last segment of the url string until last slash */ public static String getLowerPath(final String uriString) {....}
an if-case to check the requPath and esacpe the call in related.tagx? any thoughts?
-
commited a rather dirty solution to the branch '36-group-related-tags' but it works now
-
I find your solution too dirty because if someone tags a post with relevantfor the wrong url is used. You should introduce an optional attribute to the tagx file and only set it in the relavantfor.jspx.
Can you also check in the new Bootstrap layout if the same error appears? Thanks
-
Alright, thanks for the heads up.. i added an attribute to the tagx and noticed that the same problem appeared for the multiple tags, that's why i check for the '/' - count and just add the tag with '+' would you look at the merge again please?
i haven't checked for the new layout yet but it most probably will be there too. i will answer here then
-
some remarks:
- change the attribute
relevantfor
to boolean - set only the attribute to true if no other tags are requested (this should fix the links on
/relevantfor/group/GROUP_NAME/TAG
)- please check on this page the '+' - Link
- disable the + sign only when no other tags are requested i.e. on
/relevantfor/group/GROUP_NAME
The fix must be applied to the new layout as well.
- change the attribute
-
- changed version to 2.0.47
-
remarks have been implemented to the old and new layout. only the 'set only the attribute to true if no other tags are requested.. ' remark was done within the related.tagx file because of the 'getLowerPath()' function [cuts url after the third slash]
-
-
have you had the time to look over my implementation concerning your remarks?
-
- changed status to resolved
thanks!
- Log in to comment