Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cocoon.apache.org
Received-SPF: neutral (herse.apache.org: local policy)
Message-ID: <454F9BCE.7010805@apache.org>
Date: Mon, 06 Nov 2006 21:32:14 +0100
From: Giacomo Pati <giacomo@apache.org>
Organization: Apache Software Foundation
User-Agent: Thunderbird 1.5.0.7 (X11/20060909)
MIME-Version: 1.0
To: dev@cocoon.apache.org
Subject: Re: Restructuring directory structure[was [Vote] Block artifact
 directory
 structure]
References: <45470272.1030905@apache.org>
 <F27118F9-2564-4BF8-8CBE-9B9C4E69292E@trieloff.net>
 <4549ABBE.2000601@apache.org> <454BB06A.5040400@apache.org>
 <454BC985.30705@apache.org> <454BCBDB.8000807@apache.org>
 <454C8223.2040301@mobilebox.pl> <454C918F.8070504@apache.org>
 <454C9738.1070801@mobilebox.pl> <454CA68F.2050505@apache.org>
 <454CB2F5.6060107@mobilebox.pl> <454CC197.8080701@apache.org>
 <454F05F7.2070003@otego.com> <454F07DE.3020906@mobilebox.pl>
 <454F0E5A.1050701@mobilebox.pl> <454F3985.7030207@otego.com>
 <454F436B.6040808@mobilebox.pl> <454F4965.5060405@apache.org>
 <454F4B5D.2080706@apache.org> <454F4BF5.60309@mobilebox.pl>
 <454F4D9D.1060808@apache.org> <454F939E.1010403@apache.org>
 <454F9716.5030606@mobilebox.pl>
In-Reply-To: <454F9716.5030606@mobilebox.pl>
Content-Type: text/plain; charset=ISO-8859-15
Content-Transfer-Encoding: 7bit
X-DCC-spamcheck-01.tornado.cablecom.ch-Metrics: smtp-05.tornado.cablecom.ch
 1377;
	Body=1 Fuz1=1 Fuz2=1
X-Virus-Checked: Checked by ClamAV on apache.org

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


Leszek Gawron wrote:
> Giacomo Pati wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>>
>>
>> Carsten Ziegeler wrote:
>>> Leszek Gawron wrote:
>>>> Carsten Ziegeler wrote:
>>>>> Sorry, I got totally lost in all these mails. As far as I
>>>>> understand we
>>>>> are currently discussing when/where it's possible to set the
>>>>> running mode.
>>>>> There are two places: applicationContext.xml and system property. The
>>>>> system property takes precedence over the applicationContext.xml. I'm
>>>>> not sure which role cocoon.xconf should play in this?
>>>> I think everyone means applicationContext.xml. I happend to put the
>>>> wrong name in the first place.
>>>>
>>> :) OK, fine, so is there still a problem?
>>
>> Actually, yes, the problem is still there. If I understand Leszek
>> correctly, we do have to places in code where we examine the running
>> mode with the same algorithm.
> 
> I actually have two problems:
> 
> 1. IMO cocoon components should either use Settings.getRunningMode() to
> get the current running mode or have the mode injected. Allowing
> components to determine the running mode using any algorithm (even the
> easiest one) leads to inconsistencies.

I do absolutely agree with you.

> 2. Some components (like CocoonOverridePropertyConfigurer, which has
> already been fixed) use the Settings object but happily fallback to
> default mode when settings object is not available. The result of such
> situation is that a bean that has been incorrectly dependence injected
> uses other running mode than you might think. Beans using running mode.
> should simply throw if they cannot determine one.

As Carsten pointed out in a recent mail, we do not have the "original
working code" yet in the repo (dunno where it has been gone, nor which
ever class he had in mind as "original working code"). Maybe we should
just restore it and see whether that fix our problems

> I am probably picky about this but hey - you're the ones I've learnt my
> values from :)

Sure, it's easy to have someone to blame for it ;-)

Ciao

- --
Giacomo Pati
Otego AG, Switzerland - http://www.otego.com
Orixo, the XML business alliance - http://www.orixo.com

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)

iD4DBQFFT5vOLNdJvZjjVZARApFSAJUWHDjGtaxdiTWATKdplOp2vy99AJsEwPYV
LNMscahUH0/+UUGFtpWiTA==
=NIFB
-----END PGP SIGNATURE-----