Re: svn commit: r603302 - /incubator/sling/trunk/microsling/microsling-core/src/main/java/org/apache/sling/microsling/scripting/MicroslingScriptResolver.java

[Bertrand Delacretaz <bd...@apache.org>]

On Dec 11, 2007 6:50 PM, Felix Meschberger <Fe...@day.com> wrote:
> Hi,
>
> Am Dienstag, den 11.12.2007, 17:03 +0000 schrieb bdelacretaz@apache.org:
> > +
> > +        // SLING-133: do not resolve scripts for Properties, we want to use our default
> > +        // renderers for them (TODO: having that test here is really a temp fix)
> > +        if(r.adaptTo(Property.class) != null) {
> > +            return null;
> > +        }
>
> While I honestly do not have a good solution, I am always weary of such
> special case handling....

I agree that it's somewhat ugly.

>
> But then: Why should such resources not be handled by any script (or
> servlet) ?...

I'll try to explain that in my upcoming "chickens, eggs and stars"
post (how's that for buzz building? ;-)

-Bertrand

Re: svn commit: r603302 - /incubator/sling/trunk/microsling/microsling-core/src/main/java/org/apache/sling/microsling/scripting/MicroslingScriptResolver.java

[Felix Meschberger <fm...@gmail.com>]

Hi,

Am Mittwoch, den 12.12.2007, 16:18 +0100 schrieb Bertrand Delacretaz:
> > But then: Why should such resources not be handled by any script (or
> > servlet) ?...
> 
> I'll try to explain that in my upcoming "chickens, eggs and stars"
> post (how's that for buzz building? ;-)

Looking forward to that one !

Regards
Felix