Re: [Maypole] bug initialising display_tables

From: Sebastian Riedel (sri at oook.de)
Date: Mon Oct 25 2004 - 15:53:07 BST


Dave Howorth:
> Great work on the new release of Maypole :)
>
> I'm just going through modifying my application to conform to the new
> classes and thought I'd note down any issues as I found them.
>
> I think I've found a bug in Maypole.pm, where it sets display_tables:
>
> $config->display_tables
> # || $config->display_tables( [ $class->config->tables ] );
> || $config->display_tables( $class->config->tables );

Fixed.

> Maypole::Config
> ---------------
> There are a few methods/attributes/properties where there's apparently a
> discrepancy between the code and docs:
>
> application_name, tables and table_to_class are not described.
>
> document_encoding, content_type and models are present in the code but
> probably shouldn't be?

Documentation added.

>
> Is there any significance in the order in which methods are listed? If
> so, it would be good to explain it. If not, alphabetical order would be
> nice :)

Patches welcome ;)

sebastian

_______________________________________________
maypole mailing list
maypole at lists.netthink.co.uk
http://lists.netthink.co.uk/listinfo/maypole



This archive was generated by hypermail 2.1.3 : Thu Feb 24 2005 - 22:25:56 GMT