Mailing-List: contact bloodhound-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: bloodhound-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of olemis@gmail.com designates
 209.85.210.175 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAJSKAsjDjoEz7n9zG7bfB79kC2n_DKfGQSdhYW1GOQfrUAuYkQ@mail.gmail.com>
References: 
 <CAGMZAuNS3N83cSxxos-VnYk5yJj3ByY3q9XDeDnQebden7DJUA@mail.gmail.com>
	<CAJSKAsjDjoEz7n9zG7bfB79kC2n_DKfGQSdhYW1GOQfrUAuYkQ@mail.gmail.com>
Date: Mon, 17 Dec 2012 13:08:59 -0500
Message-ID: 
 <CAGMZAuO4DLawehYNAbSzhv2vXo8qZoLwR5s+m9DwfxGe=TCcUQ@mail.gmail.com>
Subject: Re: [Whitelabeling] Quick assessment of (current) whitelabelling
 implementation .
From: Olemis Lang <olemis@gmail.com>
To: bloodhound-dev@incubator.apache.org
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On 12/17/12, Peter Ko=C5=BEelj <peter@digiverse.si> wrote:
> On 13 December 2012 22:45, Olemis Lang <olemis@gmail.com> wrote:
>
>> Subjects to be discussed (IRC summary)
>>
>>   1. whethet it's better to use c =3D self.env.config ;
>> application_short =3D c.get('labels', 'application_short', "Bloodhound")
>> ; ... instead of trac.config.options
>>
>
> I have evaluated the config.options and as elegant as that is, there is n=
o
> way to specify defaults that way.

No defaults ? Since the very first time I used Trac I noticed this

{{{
#!py

class Option(object):
    """Descriptor for configuration options."""

[...]

    def __init__(self, section, name, default=3DNone, doc=3D'',
                 doc_domain=3D'tracini'):
        """Create the configuration option.

        @param section: the name of the configuration section this option
            belongs to
        @param name: the name of the option
        @param default: the default value for the option
        @param doc: documentation of the option
        """
}}}

Notice {{{default}}} argument. Isn't it just enough ? Am I missing somethin=
g ?

> That would mean that we would have to
> change trac.ini during upgrade of older installations and is not as robus=
t
> agains users mistakes when changing the config file manually.
>

If my comment above is accurate : no need to do nothing of this . This
is happening every day , every where in Trac

Maybe I didn't understand the initial statement about option defaults . CMI=
IW

>>   2. on the subject of where to put labels data maybe it should be
>> made available in request callbacks (just like req.chrome works at the
>> moment ;)
>>
>
> BH uses it's own templates from bloodhound_them together with original Tr=
ac
> request handlers. Current approach works for this combination without a
> need to change Trac codebase.
>

Believe me , I know ... and this has nothing to do with my suggestion
. I never even thought of patching Trac in any way . So , please
re-read my suggestion above and review in detail request chrome and
callbacks purpose and lifecycle .

> But we could move the labels dict from request.chrome to data inside
> BloodhoundTheme.post_process_request():
>

No .

> data['labels'] =3D self._get_whitelabelling()
> if that is considered as a more appropriate solution.
>

data is not always a dictionary ... and you might be overwriting real
data provided by a real request handler .

>
>>   3. there's another observation and it is to instantiate req. chrome
>> inside theme's post_process_request post only if
>>       * bloodhound theme is enabled and active
>>       * request handler actually needs it ... e.g. chrome should be
>>          useless most of the time for /rpc requests and alike .
>>          That's exactly why instantiation is lazy in first place .
>>
>>
> Sounds sensible but isn't "chrome" a Trac thingy?

well , indeed ... 90% is a Trac thing ;)

> AFAICT it is already
> there when request lifecycle methods form BloodhoundTheme are called.

yes and no ... it's lazy =3D> instantiated on demand .

> Does
> moving labels from "chrome" to "data" help here in any way?
>

Please review the implementation of trac.web.chrome.Chrome.dispatch
method and request callbacks . Using data may lead to clashes with
other active request handlers

--=20
Regards,

Olemis.

Blog ES: http://simelo-es.blogspot.com/
Blog EN: http://simelo-en.blogspot.com/

Featured article: