]> git.decadent.org.uk Git - maypole.git/commitdiff
fix to bug 23722
authorAaron Trevena <aaron.trevena@gmail.com>
Thu, 30 Nov 2006 16:46:12 +0000 (16:46 +0000)
committerAaron Trevena <aaron.trevena@gmail.com>
Thu, 30 Nov 2006 16:46:12 +0000 (16:46 +0000)
git-svn-id: http://svn.maypole.perl.org/Maypole/trunk@553 48953598-375a-da11-a14b-00016c27c3ee

Changes
lib/Maypole.pm
lib/Maypole/Model/CDBI/AsForm.pm

diff --git a/Changes b/Changes
index 276835e8d2613241dfdd059359ef937741773e61..dee2dbc4c6f423762e906ff39be91a9fcd06e42a 100644 (file)
--- a/Changes
+++ b/Changes
@@ -21,6 +21,7 @@ For information about current developments and future releases, see:
    fix to path handling in mod_perl and CGI when location ends in /
    fixed template path ordering so i.e. /tablename/list is used before /list when provided with a tablename
    fixed template path with array refs
+   fix to template being reset from path in plain templates (i.e. where no model), may affect those relying on the bug ( bug 23722 )
 
 2.11 Mon 31 July 2006
 
index bc41335cd5758aa76565f18c8670f29f2f5acff9..6b40b7052b5ee5e039a61c90c283b3a86340ec21 100644 (file)
@@ -638,13 +638,13 @@ sub __setup_plain_template
     # It's just a plain template
     $self->model_class(undef);
 
-    # FIXME: this is likely to be redundant and is definately causing problems.
-
-    my $path = $self->path;
-    $path =~ s{/$}{};    # De-absolutify
-    $self->path($path);
-
-    $self->template($self->path);
+    unless ($self->template) {
+      # FIXME: this is likely to be redundant and is definately causing problems.
+      my $path = $self->path;
+      $path =~ s{/$}{};    # De-absolutify
+      $self->path($path);
+      $self->template($self->path);
+    }
 }
 
 # The model has been processed or skipped (if is_applicable returned false), 
index 06f1039f3f1da76866a975733e63dbd0efb9909b..cad330c1193ac5961aebd9055780d1edeffc6e62 100644 (file)
@@ -1179,7 +1179,7 @@ sub _options_from_objects {
   for my $object (@$items) {
     my $stringify = $args->{stringify};
     if ($object->can('stringify_column') ) {
-      $stringify ||= $object->stringify_column if ($object->can($object->stringify_column));
+      $stringify ||= $object->stringify_column if ($object->stringify_column && $object->can($object->stringify_column));
     }
     my $id = $object->id;
     my $opt = HTML::Element->new("option", value => $id);