[ 
https://issues.apache.org/jira/browse/GERONIMODEVTOOLS-573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12738856#action_12738856
 ] 

Rodger Zhang commented on GERONIMODEVTOOLS-573:
-----------------------------------------------

When first click the "Security" page, then Click "Naming" page and add 
resource-ref(or other elements) ,the very error happens.
=============================================
Initial Security page->fillBody()->new 
SecurityRoleMappingSection()->createClient()->viewer.setInput(getInput())->getRoleMappings(getPlan(),
 false)->JAXBModelUtils.getSecurity(plan)
=============================================
Because JAXBModelUtils.getSecurity() set add security element to plan if the 
plan doesn't contain <security/>.
I attached a patch to modify the method getSecurity(), but I'm not sure whether 
other problems may be imported.Pls help review.

> <sec:security/> was added to plan after add a resource-ref via GEP UI
> ---------------------------------------------------------------------
>
>                 Key: GERONIMODEVTOOLS-573
>                 URL: 
> https://issues.apache.org/jira/browse/GERONIMODEVTOOLS-573
>             Project: Geronimo-Devtools
>          Issue Type: Bug
>          Components: eclipse-plugin
>    Affects Versions: 2.1.4
>         Environment: OS:suse 10 Sp2
> JDK:1.5
>            Reporter: viola.lu
>            Assignee: Tim McConnell
>            Priority: Minor
>
> Steps:
> 1.Create a dynamic web app. and edit its geronimo-web.xml via UI tool
> 2.Click "Naming" tab, and add a resource-ref: mail/TestMailSession and save 
> it.
> 3.Open "source" tab, an extral <sec:security/> is appended to 
> geronimo-web.xml, and error exists.Must remove it manually from it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to