Mailing-List: contact jetspeed-dev-help@portals.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Jetspeed Developers List" <jetspeed-dev@portals.apache.org>
Received-SPF: neutral (athena.apache.org: local policy)
Message-ID: <4BBE615C.2050003@douma.nu>
Date: Fri, 09 Apr 2010 01:06:04 +0200
From: Ate Douma <ate@douma.nu>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US;
 rv:1.9.1.9pre) Gecko/20100217 Lightning/1.0b1 Shredder/3.0.3pre
MIME-Version: 1.0
To: Jetspeed Developers List <jetspeed-dev@portals.apache.org>
Subject: Re: edit_defaults
References: <u2l4a35d831004071815r5b4f020by2a512f92aa9e2ca@mail.gmail.com>
 <x2y4a35d831004072117j148c9633i287330e2bd2e1f12@mail.gmail.com>
In-Reply-To: <x2y4a35d831004072117j148c9633i287330e2bd2e1f12@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

On 04/08/2010 06:17 AM, David Sean Taylor wrote:
> A minor complication to any portlets using the Velocity Bridge with
> the edit_defaults auto switch, seems we need to patch the bridge in
> GenericVelocityPortlet.java
> And require another release of this bridge if we want to properly
> support edit_defaults auto switch .....
Maybe we should consider upgrading bridges also to use portals-pom and release using Nexus.
If we combine that will separated and independent release versions for each bridges component like we now do with APA, e.g. start defining 
versions like bridges-common-1.0.5, bridges-velocity-1.0.5 etc. in APA JIRA, and have them all have their separate svn trunk/branches/tags 
folder, we could turn out a bridges-velocity-1.0.5 (and at least bridges-common-1.0.5 for proper Maven-2 dependency resolution) pretty quickly.
Doing the ground work starting with only bridges-common and bridges-velocity probably can be done in matter of hours.
WDYT?

>
>
> ### Eclipse Workspace Patch 1.0
> #P velocity-bridge
> Index: src/java/org/apache/portals/bridges/velocity/GenericVelocityPortlet.java
> ===================================================================
> --- src/java/org/apache/portals/bridges/velocity/GenericVelocityPortlet.java	(revision
> 931771)
> +++ src/java/org/apache/portals/bridges/velocity/GenericVelocityPortlet.java	(working
> copy)
> @@ -129,7 +129,8 @@
>        */
>       public void doEdit(RenderRequest request, RenderResponse
> response) throws PortletException, IOException
>       {
> -        super.doEdit(request, response);
> +        createPortletVelocityContext(request, response);
> +    	super.doEdit(request, response);
>       }
>
>       /**
> @@ -142,7 +143,8 @@
>        */
>       public void doHelp(RenderRequest request, RenderResponse
> response) throws PortletException, IOException
>       {
> -        super.doHelp(request, response);
> +        createPortletVelocityContext(request, response);
> +    	super.doHelp(request, response);
>       }
>
>       /**
>
>
> On Wed, Apr 7, 2010 at 6:15 PM, David Sean Taylor<d.taylor@onehippo.com>  wrote:
>> I would like to propose a simple change for the 2.2.1 release, making
>> this setting true by default in jetspeed.properties:
>>
>> # switch edit_defaults mode to edit mode automatically for portlets
>> not supporting edit_defaults mode
>> supported.portletmode.autoswitch.edit_defaults=true
>>
>> This will allow me to configure portlets for *all* users more easily,
>> not just for the current user, which seems to be what i need as
>> administrator 9 times out of 10
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org