![]() |
|
|
|
|
Change By:
|
Tobias Mattsson
(04/Nov/13 5:04 PM)
|
|
Description:
|
When a link with request parameters is clicked in the page editor the url is suffixed with ?mgnlChannel=desktop&mgnlPreview=true. It always starts with a question mark. If there's already request parameters present in the url this will fail making ?mgnlChannel=desktop part of the _value_ of the last request parameter already in the url.
Steps to reproduce:
In firefox:
* Start the pages app
* Double-click demo-features
* Click Special templates
* Click Search Result in the left navigation
* Switch to preview mode
* Enter 'a' in the search field and press enter
* A search result is presented with 21 pages
* Click to change to page 2
* The search result is now empty
The reason is that the
page
_currentPage_
parameter sent to the server is:
currentPage=2
_2
?mgnlChannel=
desktop
desktop_
The full url is:
http://localhost:8080/magnoliaAuthor/demo-features/special-templates/search-result.html?queryStr=a¤tPage=2?mgnlChannel=desktop&mgnlPreview=true
I believe the problem to be in PageEditorConnetor#init, in this block:
{code}
eventBus.addHandler(FrameNavigationEvent.TYPE, new FrameNavigationEventHandler() {
@Override
public void onFrameUrlChanged(FrameNavigationEvent frameUrlChangedEvent) {
final String platformId = getState().parameters.getPlatformType().getId();
final boolean isPreview = getState().parameters.isPreview();
final String queryString = "?mgnlChannel=" + platformId + "&mgnlPreview=" + isPreview;
view.setUrl(frameUrlChangedEvent.getPath() + queryString);
}
});
{code}
|
|
|
|
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
|
----------------------------------------------------------------
For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com>
----------------------------------------------------------------
