Return-Path: 
 <dev-return-69948-apmail-geronimo-dev-archive=geronimo.apache.org@geronimo.apache.org>
Delivered-To: apmail-geronimo-dev-archive@www.apache.org
Received: (qmail 58532 invoked from network); 29 Nov 2008 21:49:38 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2)
  by minotaur.apache.org with SMTP; 29 Nov 2008 21:49:38 -0000
Received: (qmail 61064 invoked by uid 500); 29 Nov 2008 21:49:48 -0000
Delivered-To: apmail-geronimo-dev-archive@geronimo.apache.org
Received: (qmail 61013 invoked by uid 500); 29 Nov 2008 21:49:48 -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 60997 invoked by uid 99); 29 Nov 2008 21:49:48 -0000
Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136)
    by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 29 Nov 2008 13:49:48 -0800
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; Sat, 29 Nov 2008 21:48:29 +0000
Received: from brutus (localhost [127.0.0.1])
	by brutus.apache.org (Postfix) with ESMTP id 535BA234C296
	for <dev@geronimo.apache.org>; Sat, 29 Nov 2008 13:48:46 -0800 (PST)
Message-ID: <1825072978.1227995326326.JavaMail.jira@brutus>
Date: Sat, 29 Nov 2008 13:48:46 -0800 (PST)
From: "Jarek Gawor (JIRA)" <jira@apache.org>
To: dev@geronimo.apache.org
Subject: [jira] Resolved: (GERONIMO-4407) Dupldate attribute entries exist
 in the config.xml while changing the value between zero-length string and
 no-zero-length string
In-Reply-To: <1516978723.1226541464206.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
X-Virus-Checked: Checked by ClamAV on apache.org


     [ https://issues.apache.org/jira/browse/GERONIMO-4407?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jarek Gawor resolved GERONIMO-4407.
-----------------------------------

       Resolution: Fixed
    Fix Version/s: 2.2
                   2.1.4
         Assignee: Jarek Gawor

Committed fixes for this problem to trunk (revision 721726) and branches/2.1 (revision 721729). The better solution for this problem would be to rewrite the code to use one map for attributes (instead of 3) and one map for references (instead of 2). 


> Dupldate attribute entries exist in the config.xml while changing the value between zero-length string and no-zero-length string
> --------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4407
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4407
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: kernel
>    Affects Versions: 2.1.3, 2.2
>         Environment: Windows XP
> JDK 1.5.0
>            Reporter: Ivan
>            Assignee: Jarek Gawor
>             Fix For: 2.1.4, 2.2
>
>
> While an attribute's initial value is no-zero length  value, then we set  its value with zero-length value in the portlet, then we could find the in the config.xml file, there are  duplicate attribute entries.
> Like 
>             .......
>             <attribute name="LoginTimeout">0</attribute>
>             <attribute name="ShutdownDatabase">shutdown2</attribute>
>             <attribute name="ShutdownDatabase"/>
>             ......
> It seems that it is caused we use two Map (attributes, clearAttributes ) to keep the attribute values, while we setting the attribute, no code checks that the attribute should only exist in one map. 

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