[html-formfu] [PATCH] bug with multiple config_file_path in _load_config

[Carl Franks]

Hi Adam,

The behaviour you describe that you want, sounds correct.
The patch looks good.
I'll commit it soon, or if you have a google account, send me your
associated email address, and I'll give you commit.

Cheers,
Carl


On 10 November 2010 17:59, Adam Prime <aprime at oanda.com> wrote:
> Does anyone have any opinion on this?  We need this feature really badly, to
> the point where we're going to have to do a local custom package of the
> current release version that has it in, but i would very much like to know
> that it's going to make it into formfu, and that we'll eventually be able to
> get back to using the normal cpan package.
>
> Also, if you don't understand what i'm talking about, I can provide a more
> explicit example of the use case, just let me know.
>
> Adam
>
> On 10-11-04 12:45 PM, Adam Prime wrote:
>
> If you have a form object with multiple config_file_paths, load_config_file
> (etc) don't behave in the same way as TT does when it has multiple
> directories in it's INCLUDE_PATH.  I assume, possibly incorrectly, that that
> it should.  What it does right now is take the list of directories, iterate
> through them attempting to load your config file, and if it does it happily
> continues through the list of paths, so if it finds another one, it'll use
> that one instead.  TT uses the first one it finds, which allows you to have
> customized version of shared templates by customizing INCLUDE_PATH.  We need
> to do the same thing with FormFu at $work, which is how we've run into this
> problem.
>
> Attached is an svn diff for a new test which fails in the current version,
> and the corresponding fix for the problem.
>
> Let me know if you need any clarification, or anything else, and thanks for
> the great package, it's save us tremendous amounts of time.
>
> Adam
>
> _______________________________________________
> HTML-FormFu mailing list
> HTML-FormFu at lists.scsys.co.uk
> http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu
>
> _______________________________________________
> HTML-FormFu mailing list
> HTML-FormFu at lists.scsys.co.uk
> http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu
>