Bug 984472 - [Permanent fix] Backport of GTNPORTAL-3182, GTNPORTAL-3156 and GTNPORTAL-3246
Summary: [Permanent fix] Backport of GTNPORTAL-3182, GTNPORTAL-3156 and GTNPORTAL-3246
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: JBoss Enterprise Portal Platform 6
Classification: JBoss
Component: Portal
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
high
unspecified
Target Milestone: DR01
: 6.1.1
Assignee: Peter Palaga
QA Contact:
URL:
Whiteboard: FixAvail
Depends On: 984405
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-15 10:29 UTC by Adam Kovari
Modified: 2025-02-10 03:28 UTC (History)
5 users (show)

Fixed In Version:
Clone Of: 984405
Environment:
- JPP 6.0
Last Closed: 2025-02-10 03:28:05 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker GTNPORTAL-3156 0 Minor Resolved NullPointerException when rendering portlet in method UIPortlet.getSuitedTheme() 2014-02-19 16:37:05 UTC
Red Hat Issue Tracker GTNPORTAL-3182 0 Major Resolved Portal 'overriding' maximization of a window in case multiple portlets on the page receive any event but only one of the... 2014-02-19 16:37:07 UTC
Red Hat Issue Tracker GTNPORTAL-3246 0 Major Resolved the only maximized portlet state of last page is kept 2014-02-19 16:37:03 UTC

Description Adam Kovari 2013-07-15 10:29:50 UTC
+++ This bug was initially created as a clone of Bug #984405 +++

This is for permanent fix for GTNPORTAL-3182 for JPP 6.x

Comment 1 JBoss JIRA Server 2013-07-19 13:03:09 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3182

Reproducer portlet

Comment 3 JBoss JIRA Server 2013-07-19 13:04:54 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3182

Configuration for the reproducer

Comment 4 JBoss JIRA Server 2013-07-22 07:23:55 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3182

I have added some more code to make sure that maximization of multiple portlets one after another works properly when done using AJAX code. This works fine as it triggers an Action request.

Comment 5 JBoss JIRA Server 2013-07-23 12:34:01 UTC
Adam Kovari <akovari> updated the status of jira GTNPORTAL-3182 to Reopened

Comment 6 JBoss JIRA Server 2013-07-23 12:34:01 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3182

Sending new pull request: https://github.com/gatein/gatein-portal/pull/594

Comment 7 JBoss JIRA Server 2013-08-09 09:56:04 UTC
Trong Tran <trongtt> made a comment on jira GTNPORTAL-3182

There is still another small problem with Portlet maximization :

1. Click on the maximize icon of a portlet in Home page
2. Switch to Sitemap page
3. Switch back to Home page. The previously maximized portlet is restore to Normal ==> NOK

Comment 8 JBoss JIRA Server 2013-08-09 10:13:26 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3182

Let me take a look at that too. Thanks

Comment 9 JBoss JIRA Server 2013-08-09 13:35:04 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3182

I have fixed this in UIPortal.refreshPage() by commenting out code that sets null to maximized component. BTW I noticed UIPortal has again maximized component field, so now there are 3 places - UIPage,UIPageBody and UIPortal, maybe this should be unified.
Another problem I discovered was minimizing portlet that got maximized from the producer portlet using event before. This should work now as well.

Comment 10 JBoss JIRA Server 2013-08-09 16:32:20 UTC
Trong Tran <trongtt> made a comment on jira GTNPORTAL-3182

UIPortal maintains a different maximized component which is used to show full portal space. BUT seems now it doesn't work :(

Comment 11 JBoss JIRA Server 2013-08-09 18:03:09 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3182

Could you be more specific? I tried virtually any window state change I could think of and it worked fine.

Comment 12 JBoss JIRA Server 2013-08-11 16:31:11 UTC
Trong Tran <trongtt> made a comment on jira GTNPORTAL-3182

I mean a portlet in Portal layout can be maximized, just like in Page layout

Comment 13 JBoss JIRA Server 2013-08-11 16:43:27 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3182

Hmm, so do you think PR 609 is now safe to merge? I guess UIPortal maximized component is somewhat unrelated to this JIRA anyway.

Comment 14 JBoss JIRA Server 2013-08-12 03:55:25 UTC
Trong Tran <trongtt> made a comment on jira GTNPORTAL-3182

Yes, I think the PR is ok to merge.

I just created another JIRA for maximized component in Portal layout GTNPORTAL-3233

Comment 15 JBoss JIRA Server 2013-08-19 13:08:07 UTC
Adam Kovari <akovari> made a comment on jira GTNPORTAL-3156

Updated the PR based on Tran's suggestions.

Comment 16 JBoss JIRA Server 2013-08-21 11:14:45 UTC
Trong Tran <trongtt> made a comment on jira GTNPORTAL-3156

The Fix Version/s is not set properly, it should be 3.7.0.Final (master) instead

Comment 17 JBoss JIRA Server 2013-09-10 11:13:40 UTC
Peter Palaga <ppalaga> made a comment on jira GTNPORTAL-3246

One more related PR was merged: https://github.com/gatein/gatein-portal/pull/640

Comment 18 Jared MORGAN 2014-02-04 22:26:07 UTC
If this issue requires mention in release notes (as indicated by the requires_doc_text flag) draft content must be provided by the engineer responsible. 

Please complete the Doc Text field and supply draft info. Docs can do any edits required and mark the issue for inclusion in the Release Notes for RHJP 6.1.1

Comment 19 Peter Palaga 2014-02-06 16:55:09 UTC
I cannot set requires_doc_text	to +.

Comment 20 Jared MORGAN 2014-02-06 20:30:31 UTC
(In reply to Peter Palaga from comment #19)
> I cannot set requires_doc_text	to +.

Hi Peter

Thanks for the CCFR text. 

You don't need to worry about setting the flag to + - this is done by the docs team after we've reviewed it. 

However if you can't set the flag to minus, that could be a reason why we're seeing all these bugs without any flag status set. Devs may simply not be able to set the flag (could you check if minus is something you can set)?

Comment 23 Red Hat Bugzilla 2025-02-10 03:28:05 UTC
This product has been discontinued or is no longer tracked in Red Hat Bugzilla.


Note You need to log in before you can comment on or make changes to this bug.