[andre]

Есть какие-либо правила на предложения в изменении движка?

У меня есть предложение: файл includes/functions/html_output.php функция: osc_draw_file_field.


Назначение: Добавить возможность выставления доп. параметров, что бы выдержать интерфейс в одном стиле.

Новый текст функции:

Код: [Выделить]

/**
 * Outputs a form file upload field
 *
 * @param string $name The name and ID of the file upload field
 * @param boolean $show_max_size Show the maximum file upload size beside the field
 * @param string $parameters Additional parameters for the file field
 * @access public
 */

  function osc_draw_file_field($name, $show_max_size = false, $param = null) {
    global $osC_Language;

    static $upload_max_filesize;

    if (!is_bool($show_max_size)) {
      $show_max_size = false;
    }

    $field = osc_draw_input_field($name, null, $param, false, 'file');

    if ($show_max_size === true) {
      if (!isset($upload_max_filesize)) {
        $upload_max_filesize = @ini_get('upload_max_filesize');
      }

      if (!empty($upload_max_filesize)) {
        $field .= ' ' . sprintf($osC_Language->get('maximum_file_upload_size'), osc_output_string($upload_max_filesize));
      }
    }

    return $field;
  }

[musicman]

Поподробнее пожалуйста....

[andre]

Я сделал изменение в core и, как мне кажется, они будут полезны всем. Есть ли общепризнанный механизм сбора таких изменений?

[musicman]

Любые изменения, которые улучшат 3-ю версию приемлимы... Расскажите на что влияют Ваши изменения и как это улучшает на практике сам магазин... Вполне возможно что в сервиспаке Ваши правки могут быть полезны для всех нас.

[andre]

Любые изменения, которые улучшат 3-ю версию приемлимы... Расскажите на что влияют Ваши изменения и как это улучшает на практике сам магазин... Вполне возможно что в сервиспаке Ваши правки могут быть полезны для всех нас.

файл includes/functions/html_output.php
функция: osc_draw_file_field
Цель улучшения: Возможность выставления дополнительных параметров при отрисовке input'a типа file.
Описание: при разработке модуля по загрузке прайс-листа столкнулся с проблемой, что функция не поддерживает установку дополнительных параметров, поэтому нет возможности выдержать интерфейс в одном стиле.
Предложение: Добавить параметр, по аналогии с другими функциями из данного модуля
Измененный текст функции:

Код: [Выделить]

/**
 * Outputs a form file upload field
 *
 * @param string $name The name and ID of the file upload field
 * @param boolean $show_max_size Show the maximum file upload size beside the field
 * @param string $parameters Additional parameters for the file field
 * @access public
 */

  function osc_draw_file_field($name, $show_max_size = false, $param = null) {
    global $osC_Language;

    static $upload_max_filesize;

    if (!is_bool($show_max_size)) {
      $show_max_size = false;
    }

    $field = osc_draw_input_field($name, null, $param, false, 'file');

    if ($show_max_size === true) {
      if (!isset($upload_max_filesize)) {
        $upload_max_filesize = @ini_get('upload_max_filesize');
      }

      if (!empty($upload_max_filesize)) {
        $field .= ' ' . sprintf($osC_Language->get('maximum_file_upload_size'), osc_output_string($upload_max_filesize));
      }
    }

    return $field;
  }