[jira] [Updated] (ISIS-909) BookmarksPagesPanel often cannot detect the current/active model
[ https://issues.apache.org/jira/browse/ISIS-909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dan Haywood updated ISIS-909: - Fix Version/s: (was: 2.3.0) 1.20.0 Description: See also ISIS-842. ~~~ There is a logic [1] at BookmarkedPagesPanel.java that tries to detect whether an item in the bookmarks sliding menu is the currently viewed/edited item and sets CSS class "currentBookmark". Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. It uses Objects.equal(current, pageParameters) and often those have similar oid but additional parameters make the check failing. 1. [https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167] was: There is a logic [1] at BookmarkedPagesPanel.java that tries to detect whether an item in the bookmarks sliding menu is the currently viewed/edited item and sets CSS class "currentBookmark". Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. It uses Objects.equal(current, pageParameters) and often those have similar oid but additional parameters make the check failing. 1. https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167 > BookmarksPagesPanel often cannot detect the current/active model > > > Key: ISIS-909 > URL: https://issues.apache.org/jira/browse/ISIS-909 > Project: Isis > Issue Type: Bug > Components: Core: Viewer: Wicket >Affects Versions: viewer-wicket-1.6.0 >Reporter: Martin Grigorov >Priority: Minor > Fix For: 1.20.0 > > > See also ISIS-842. > > ~~~ > There is a logic [1] at BookmarkedPagesPanel.java that tries to detect > whether an item in the bookmarks sliding menu is the currently viewed/edited > item and sets CSS class "currentBookmark". > Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. > It uses Objects.equal(current, pageParameters) and often those have similar > oid but additional parameters make the check failing. > 1. > [https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167] -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (ISIS-909) BookmarksPagesPanel often cannot detect the current/active model
[ https://issues.apache.org/jira/browse/ISIS-909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dan Haywood updated ISIS-909: - Fix Version/s: (was: 1.16.0) 1.14.0 > BookmarksPagesPanel often cannot detect the current/active model > > > Key: ISIS-909 > URL: https://issues.apache.org/jira/browse/ISIS-909 > Project: Isis > Issue Type: Bug > Components: Core: Viewer: Wicket >Affects Versions: viewer-wicket-1.6.0 >Reporter: Martin Grigorov >Assignee: Dan Haywood >Priority: Minor > Fix For: 1.14.0 > > > There is a logic [1] at BookmarkedPagesPanel.java that tries to detect > whether an item in the bookmarks sliding menu is the currently viewed/edited > item and sets CSS class "currentBookmark". > Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. > It uses Objects.equal(current, pageParameters) and often those have similar > oid but additional parameters make the check failing. > 1. > https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (ISIS-909) BookmarksPagesPanel often cannot detect the current/active model
[ https://issues.apache.org/jira/browse/ISIS-909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dan Haywood updated ISIS-909: - Fix Version/s: (was: 1.17.0) 1.14.0 > BookmarksPagesPanel often cannot detect the current/active model > > > Key: ISIS-909 > URL: https://issues.apache.org/jira/browse/ISIS-909 > Project: Isis > Issue Type: Bug > Components: Core: Viewer: Wicket >Affects Versions: viewer-wicket-1.6.0 >Reporter: Martin Grigorov >Assignee: Dan Haywood >Priority: Minor > Fix For: 1.14.0 > > > There is a logic [1] at BookmarkedPagesPanel.java that tries to detect > whether an item in the bookmarks sliding menu is the currently viewed/edited > item and sets CSS class "currentBookmark". > Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. > It uses Objects.equal(current, pageParameters) and often those have similar > oid but additional parameters make the check failing. > 1. > https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (ISIS-909) BookmarksPagesPanel often cannot detect the current/active model
[ https://issues.apache.org/jira/browse/ISIS-909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dan Haywood updated ISIS-909: - Fix Version/s: (was: core-1.9.0) core-1.10.0 > BookmarksPagesPanel often cannot detect the current/active model > > > Key: ISIS-909 > URL: https://issues.apache.org/jira/browse/ISIS-909 > Project: Isis > Issue Type: Bug > Components: Core: Viewer: Wicket >Affects Versions: viewer-wicket-1.6.0 >Reporter: Martin Grigorov >Assignee: Dan Haywood >Priority: Minor > Fix For: core-1.10.0 > > > There is a logic [1] at BookmarkedPagesPanel.java that tries to detect > whether an item in the bookmarks sliding menu is the currently viewed/edited > item and sets CSS class "currentBookmark". > Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. > It uses Objects.equal(current, pageParameters) and often those have similar > oid but additional parameters make the check failing. > 1. > https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (ISIS-909) BookmarksPagesPanel often cannot detect the current/active model
[ https://issues.apache.org/jira/browse/ISIS-909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dan Haywood updated ISIS-909: - Fix Version/s: core-1.8.0 > BookmarksPagesPanel often cannot detect the current/active model > > > Key: ISIS-909 > URL: https://issues.apache.org/jira/browse/ISIS-909 > Project: Isis > Issue Type: Bug > Components: Viewer: Wicket >Affects Versions: viewer-wicket-1.6.0 >Reporter: Martin Grigorov >Assignee: Dan Haywood >Priority: Minor > Fix For: core-1.8.0 > > > There is a logic [1] at BookmarkedPagesPanel.java that tries to detect > whether an item in the bookmarks sliding menu is the currently viewed/edited > item and sets CSS class "currentBookmark". > Most of the time bookmarkedPagesModel.isCurrent(pageParameters) doesn't work. > It uses Objects.equal(current, pageParameters) and often those have similar > oid but additional parameters make the check failing. > 1. > https://github.com/apache/isis/blob/master/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/components/bookmarkedpages/BookmarkedPagesPanel.java#L167 -- This message was sent by Atlassian JIRA (v6.3.4#6332)