You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@vcl.apache.org by jf...@apache.org on 2012/11/15 22:05:00 UTC
svn commit: r1410011 - /vcl/trunk/web/.ht-inc/images.php
Author: jfthomps
Date: Thu Nov 15 21:04:59 2012
New Revision: 1410011
URL: http://svn.apache.org/viewvc?rev=1410011&view=rev
Log:
VCL-628
check for duplicate image name not correctly performed when specified name is the same as the base image
images.php: modified processImageInput - pass empty value for $imageid when calling checkForImageName if adding a new image
Modified:
vcl/trunk/web/.ht-inc/images.php
Modified: vcl/trunk/web/.ht-inc/images.php
URL: http://svn.apache.org/viewvc/vcl/trunk/web/.ht-inc/images.php?rev=1410011&r1=1410010&r2=1410011&view=diff
==============================================================================
--- vcl/trunk/web/.ht-inc/images.php (original)
+++ vcl/trunk/web/.ht-inc/images.php Thu Nov 15 21:04:59 2012
@@ -1121,7 +1121,7 @@ function editOrAddImage($state) {
print " <div align=\"center\">\n";
print " <button dojoType=\"dijit.form.Button\">\n";
print " Software is Manually Installed\n";
- print " <script type=\"dojo/method\" event=\"onClick\">\n";
+ print " <script type=\"dojo/method\" event=\"onClick\">\n";
print " dijit.byId('autoconfirmdlg').hide();\n";
print " addConnectMethod3();\n";
print " dijit.byId('connectmethoddlg').openDropDown();\n";
@@ -1129,7 +1129,7 @@ function editOrAddImage($state) {
print " </button>\n";
print " <button dojoType=\"dijit.form.Button\">\n";
print " Cancel\n";
- print " <script type=\"dojo/method\" event=\"onClick\">\n";
+ print " <script type=\"dojo/method\" event=\"onClick\">\n";
print " dijit.byId('autoconfirmdlg').hide();\n";
print " dijit.byId('connectmethoddlg').openDropDown();\n";
print " </script>\n";
@@ -1137,12 +1137,12 @@ function editOrAddImage($state) {
print " </div>\n";
print "</div>\n";
- if(!$state){
- print "<div id=revisiondiv>\n";
- print getRevisionHTML($data['imageid']);
- print "</div>\n";
- }
- print "</DIV>\n";
+ if(!$state){
+ print "<div id=revisiondiv>\n";
+ print getRevisionHTML($data['imageid']);
+ print "</div>\n";
+ }
+ print "</DIV>\n";
}
////////////////////////////////////////////////////////////////////////////////
@@ -3085,10 +3085,15 @@ function processImageInput($checks=1) {
$submitErrMsg[PRETTYNAMEERR] = "Name can only contain alphabets, numbers, signs, and spaces.";
}
- if(! ($submitErr & PRETTYNAMEERR) &&
- checkForImageName($return["prettyname"], "long", $return["imageid"])) {
- $submitErr |= PRETTYNAMEERR;
- $submitErrMsg[PRETTYNAMEERR] = "An image already exists with this name.";
+ if(! ($submitErr & PRETTYNAMEERR)) {
+ if($return['requestid'] == '')
+ $imageid = $return['imageid'];
+ else
+ $imageid = '';
+ if(checkForImageName($return["prettyname"], "long", $imageid)) {
+ $submitErr |= PRETTYNAMEERR;
+ $submitErrMsg[PRETTYNAMEERR] = "An image already exists with this name.";
+ }
}
if($return["minram"] < 0 || $return["minram"] > 20480) {
$submitErr |= MINRAMERR;