Browse files

Undo previous commit of automatically adding id="$name" to input fiel…

…ds; multiple fields could share the same name/id even though they would exist in different forms
  • Loading branch information...
1 parent 0f53b09 commit abf6215e7d8bb80c6d3d154c76fbf6e42042cd08 @haraldpdl haraldpdl committed Nov 12, 2010
Showing with 0 additions and 24 deletions.
  1. +0 −12 catalog/admin/includes/functions/html_output.php
  2. +0 −12 catalog/includes/functions/html_output.php
View
12 catalog/admin/includes/functions/html_output.php
@@ -176,10 +176,6 @@ function tep_draw_input_field($name, $value = '', $parameters = '', $required =
$field = '<input type="' . tep_output_string($type) . '" name="' . tep_output_string($name) . '"';
- if (strpos($parameters, 'id=') === false) {
- $field .= ' id="' . tep_output_string($name) . '"';
- }
-
if ( ($reinsert_value == true) && ( (isset($HTTP_GET_VARS[$name]) && is_string($HTTP_GET_VARS[$name])) || (isset($HTTP_POST_VARS[$name]) && is_string($HTTP_POST_VARS[$name])) ) ) {
if (isset($HTTP_GET_VARS[$name]) && is_string($HTTP_GET_VARS[$name])) {
$value = stripslashes($HTTP_GET_VARS[$name]);
@@ -255,10 +251,6 @@ function tep_draw_textarea_field($name, $wrap, $width, $height, $text = '', $par
$field = '<textarea name="' . tep_output_string($name) . '" cols="' . tep_output_string($width) . '" rows="' . tep_output_string($height) . '"';
- if (strpos($parameters, 'id=') === false) {
- $field .= ' id="' . tep_output_string($name) . '"';
- }
-
if (tep_not_null($parameters)) $field .= ' ' . $parameters;
$field .= '>';
@@ -321,10 +313,6 @@ function tep_draw_pull_down_menu($name, $values, $default = '', $parameters = ''
$field = '<select name="' . tep_output_string($name) . '"';
- if (strpos($parameters, 'id=') === false) {
- $field .= ' id="' . tep_output_string($name) . '"';
- }
-
if (tep_not_null($parameters)) $field .= ' ' . $parameters;
$field .= '>';
View
12 catalog/includes/functions/html_output.php
@@ -170,10 +170,6 @@ function tep_draw_input_field($name, $value = '', $parameters = '', $type = 'tex
$field = '<input type="' . tep_output_string($type) . '" name="' . tep_output_string($name) . '"';
- if (strpos($parameters, 'id=') === false) {
- $field .= ' id="' . tep_output_string($name) . '"';
- }
-
if ( ($reinsert_value == true) && ( (isset($HTTP_GET_VARS[$name]) && is_string($HTTP_GET_VARS[$name])) || (isset($HTTP_POST_VARS[$name]) && is_string($HTTP_POST_VARS[$name])) ) ) {
if (isset($HTTP_GET_VARS[$name]) && is_string($HTTP_GET_VARS[$name])) {
$value = stripslashes($HTTP_GET_VARS[$name]);
@@ -239,10 +235,6 @@ function tep_draw_textarea_field($name, $wrap, $width, $height, $text = '', $par
$field = '<textarea name="' . tep_output_string($name) . '" cols="' . tep_output_string($width) . '" rows="' . tep_output_string($height) . '"';
- if (strpos($parameters, 'id=') === false) {
- $field .= ' id="' . tep_output_string($name) . '"';
- }
-
if (tep_not_null($parameters)) $field .= ' ' . $parameters;
$field .= '>';
@@ -303,10 +295,6 @@ function tep_draw_pull_down_menu($name, $values, $default = '', $parameters = ''
$field = '<select name="' . tep_output_string($name) . '"';
- if (strpos($parameters, 'id=') === false) {
- $field .= ' id="' . tep_output_string($name) . '"';
- }
-
if (tep_not_null($parameters)) $field .= ' ' . $parameters;
$field .= '>';

0 comments on commit abf6215

Please sign in to comment.