mason is more strict about variables - patch from Richard Siddall, thanks

diff --git a/Makefile b/Makefile
index cf9f6f5..07a0509 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -175,7 +175,8 @@ upload-docs: forcehtmlman
        ssh cleanwhisker.420.am rm -rf /var/www/www.sisd.com/freeside/docs
        scp -pr httemplate/docs cleanwhisker.420.am:/var/www/www.sisd.com/freeside/docs
 
-release: upload-docs update-webdemo
+#release: upload-docs update-webdemo
+release:
        cd /home/ivan/freeside1.4
        #cvs tag ${TAG}
        cvs tag -F ${TAG}

diff --git a/httemplate/edit/part_export.cgi b/httemplate/edit/part_export.cgi
index 839a114..4defbc8 100644 (file)
--- a/httemplate/edit/part_export.cgi
+++ b/httemplate/edit/part_export.cgi
@@ -60,7 +60,7 @@ my $widget = new HTML::Widgets::SelectLayers(
         foreach my $select_option ( @{$optinfo->{options}} ) {
           #if ( ref($select_option) ) {
           #} else {
-            $selected = $select_option eq $value ? ' SELECTED' : '';
+            my $selected = $select_option eq $value ? ' SELECTED' : '';
             $html .= qq!<OPTION VALUE="$select_option"$selected>!.
                      qq!$select_option</OPTION>!;
           #}