Mailing-List: contact dev-help@shale.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@shale.apache.org
Received-SPF: pass (herse.apache.org: domain of mario.buonopane@accenture.com
 designates 170.252.72.93 as permitted sender)
Content-Class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Importance: normal
Priority: normal
Content-Transfer-Encoding: quoted-printable
Subject: SHALE-409 fix
Date: Wed, 23 May 2007 14:24:15 +0200
Message-ID: 
 <7D4D7E701546E34D9BF0D1E80F76209E01DD8D9C@EMEXM0203.dir.svc.accenture.com>
Thread-Topic: SHALE-409 fix
thread-index: AcechqfZJWPkhh/7SPaPijQhyIk7YgArIrnQAAB8IdA=
From: <mario.buonopane@accenture.com>
To: <dev@shale.apache.org>


Wendy, I have tested the patch in this way:

I have modified the private method afterRenderResponse of
ViewPhaseListener class:

=20

    private void afterRenderResponse(PhaseEvent event) {

=20

        // Initialize local values we will need

        Map map =3D
event.getFacesContext().getExternalContext().getRequestMap();

        // Remove our list of initialized views explicitly

        map.remove(FacesConstants.VIEWS_INITIALIZED);

     =20

        List list =3D new ArrayList();

        Iterator entries =3D map.entrySet().iterator();

=20

=20

        // First select all the ViewController and AbstractRequestBean
instances

        while (entries.hasNext()) {

            Map.Entry entry =3D (Map.Entry) entries.next();

            if ((entry.getValue() instanceof ViewController)

             || (entry.getValue() instanceof AbstractRequestBean)) {

                list.add(entry.getKey());

            }

        }

=20

        // Second select all remaining instances, which will include
annotated

        // managed beans if Shale Tiger is present

        /* COMMENTED BY BUONOPANE

        entries =3D map.entrySet().iterator();

        while (entries.hasNext()) {

            Map.Entry entry =3D (Map.Entry) entries.next();

            if (!list.contains(entry.getKey())) {

                list.add(entry.getKey());

            }

        }*/

=20

        // Iterate through the keys in the specified order, removing the

        // corresponding request scope attribute instances

        Iterator keys =3D list.iterator();

        while (keys.hasNext()) {

            String key =3D (String) keys.next();

            try {

                  log.info("******** removing "+key);

                map.remove(key);=20

            } catch (Exception e) {

                handleException(event.getFacesContext(), e);

            }

        }

=20

    }

I have just commented the line of the second loop identified by the
string COMMENTED BY BUONOPANE.=20

But I don't know if the loop I commented need to be modified.

What do you think?
=20
Regards=20
Mario=20
 =20

-----Original Message-----
From: Wendy Smoak [mailto:wsmoak@gmail.com]=20
Sent: 22 maggio 2007 17.33
To: user@shale.apache.org
Subject: Re: SHALE-409 fix

On 5/22/07, mario.buonopane@accenture.com
<mario.buonopane@accenture.com> wrote:

> Antonio, I have tested the patch suggested by Matt Hughes at
> http://issues.apache.org/struts/browse/SHALE-409.

Great!  First, please come join us on the development list.  We
generally keep the user list for discussion about using released
versions.  You can find subscription info here, or you can post
through the Nabble forum interface:
http://shale.apache.org/mail-lists.html

What we need is a "svn diff" of your changes against the latest Shale
code.  That's the quickest way for the developers to review and apply
changes contributed by the community.

Assuming you have the Shale 1.0.x branch checked out and have made
changes to it locally, execute "svn diff > SHALE-409.patch" and attach
the patch file to the issue.

Thanks,
--=20
Wendy


This message is for the designated recipient only and may contain =
privileged, proprietary, or otherwise private information.  If you have =
received it in error, please notify the sender immediately and delete =
the original.  Any other use of the email by you is prohibited.