Bug 983063

Summary: Administration / Site Redirects - Condition table is incomplete
Product: [JBoss] JBoss Enterprise Portal Platform 6 Reporter: Dominik Pospisil <dpospisi>
Component: PortalAssignee: Default User <jbpapp-maint>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.1.0CC: bdawidow, epp-bugs, mwringe
Target Milestone: ER04   
Target Release: 6.1.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-11-07 14:21:58 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Screenshot of the site redirect conditions
none
redirect overview page none

Description Dominik Pospisil 2013-07-10 12:06:49 UTC 
Administration / Site Redirects - Condition table seems to be incomplete.

The first column show condition name and the second column shows Contain/ Does Not contain value.

However the table does not show what is the value related to - User Agent or Propery. Without this information the table does not make much sense.
Comment 2 Matt Wringe 2013-07-15 14:48:25 UTC 
Created attachment 773799 [details]
Screenshot of the site redirect conditions

I am confused over why this issue has been raise. Attached is a screen shot which shows the redirect admin.

It should be clear from the screen shot that the section under 'User Agent String' contains the settings for user agent strings while the section under 'Property' handles the javascript inspected device properties.

Note: only User Agent string uses 'contains' 'does not contain', the device properties use different options. Please see the documentation.
Comment 3 Dominik Pospisil 2013-07-15 16:04:42 UTC 
Hi Mat,

sorry for not being specific enough. I was referring to redirect configuration page, not property editor page.

The page shows settings only for user-agent based conditions without any indication that user-agent based condition is used. For property based, the settings column is left blank completely.
Comment 4 Dominik Pospisil 2013-07-15 16:05:43 UTC 
Created attachment 773818 [details]
redirect overview page
Comment 5 Matt Wringe 2013-07-15 18:37:25 UTC 
Ah, yeah, ok, makes a lot more sense now. Thanks for updating the bug with the screenshot.

Gabriel needs to make a decision on how this should be fixed (then Alex can implement the fix). Created a jira to track this on the GateIn side (GTNPORTAL-3183).
Comment 6 JBoss JIRA Server 2013-07-16 19:12:50 UTC 
Gabriel Cardoso <gcardoso> made a comment on jira GTNPORTAL-3183

After discussing with Alexandre and Matt, we agreed on the following solution: http://statichtml-theute.rhcloud.com/cardosogabriel/gatein-client/site-redirects.html

Code is available at: https://github.com/cardosogabriel/gatein-client/commit/49e69952ad7d5a1f56838d2a8e9b9ae3bb80f4bf
Comment 7 Dominik Pospisil 2013-08-21 11:00:59 UTC 
Verified (ER4).