Return-Path: 
 <dev-return-75539-apmail-geronimo-dev-archive=geronimo.apache.org@geronimo.apache.org>
Delivered-To: apmail-geronimo-dev-archive@www.apache.org
Received: (qmail 9405 invoked from network); 4 Aug 2009 06:31:42 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3)
  by minotaur.apache.org with SMTP; 4 Aug 2009 06:31:42 -0000
Received: (qmail 29632 invoked by uid 500); 4 Aug 2009 06:31:46 -0000
Delivered-To: apmail-geronimo-dev-archive@geronimo.apache.org
Received: (qmail 29520 invoked by uid 500); 4 Aug 2009 06:31:46 -0000
Mailing-List: contact dev-help@geronimo.apache.org; run by ezmlm
Precedence: bulk
list-help: <mailto:dev-help@geronimo.apache.org>
list-unsubscribe: <mailto:dev-unsubscribe@geronimo.apache.org>
List-Post: <mailto:dev@geronimo.apache.org>
Reply-To: dev@geronimo.apache.org
List-Id: <dev.geronimo.apache.org>
Delivered-To: mailing list dev@geronimo.apache.org
Received: (qmail 29458 invoked by uid 99); 4 Aug 2009 06:31:46 -0000
Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 04 Aug 2009 06:31:46 +0000
X-ASF-Spam-Status: No, hits=-2000.0 required=10.0
	tests=ALL_TRUSTED
X-Spam-Check-By: apache.org
Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 04 Aug 2009 06:31:36 +0000
Received: from brutus (localhost [127.0.0.1])
	by brutus.apache.org (Postfix) with ESMTP id 0FD6B29A0020
	for <dev@geronimo.apache.org>; Mon,  3 Aug 2009 23:31:15 -0700 (PDT)
Message-ID: <805981600.1249367475064.JavaMail.jira@brutus>
Date: Mon, 3 Aug 2009 23:31:15 -0700 (PDT)
From: "Rodger Zhang (JIRA)" <jira@apache.org>
To: dev@geronimo.apache.org
Subject: [jira] Commented: (GERONIMODEVTOOLS-573) <sec:security/> was added
 to plan after add a resource-ref via GEP UI
In-Reply-To: <1451390815.1243235805620.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394
X-Virus-Checked: Checked by ClamAV on apache.org


    [ 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.