[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-3315) seamspace example does not validate permission settings

Jay Balunas (JIRA) jira-events at lists.jboss.org
Thu Sep 18 10:58:21 EDT 2008


    [ https://jira.jboss.org/jira/browse/JBSEAM-3315?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12430230#action_12430230 ] 

Jay Balunas commented on JBSEAM-3315:
-------------------------------------

This is not a functionality problem, but a gui validation issue - you were looking too deep into a minor bug ;-)

In the gui when setting view and or comment permissions you can set a permission to comment, but not view.  This results in the user not seeing the object that they technically can comment on.  A user can also set neither view or comment.  Then when it is submitted the permission is not created, but there is no message to the use.

I would think a simple validation check in JSF can catch this.  At least one permission must be set, and one must be view.



> seamspace example does not validate permission settings 
> --------------------------------------------------------
>
>                 Key: JBSEAM-3315
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-3315
>             Project: Seam
>          Issue Type: Bug
>          Components: Examples
>    Affects Versions: 2.1.0.BETA1
>            Reporter: Jay Balunas
>            Assignee: Shane Bryzak
>            Priority: Minor
>             Fix For: 2.1.0.CR1
>
>
> minor comments:
> seamspace example does not validate permission settings.  
> This means that you can set "comment" permission, but not "view" permissions.  This results in users not able to see the image ;-)
> Also you don't have to set view or comment and it will just not create the permission when you submit - it should probably have an invalid config message.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        



More information about the seam-issues mailing list