[Not a bug] logical Error in admin/functions/html_output

[FredSchenk]

The function zen_draw_from in html_output starts with the following code:

  function zen_draw_form($name, $action, $parameters = '', $method = 'post', $params = '', $usessl = 'false') {
    $form = '<form name="' . zen_output_string($name) . '" action="';
    if (zen_not_null($parameters)) {
      if ($usessl) {
        $form .= zen_href_link($action, $parameters, 'NONSSL');
      } else {
        $form .= zen_href_link($action, $parameters, 'NONSSL');
      }
    } else {
      if ($usessl) {
        $form .= zen_href_link($action, '', 'NONSSL');
      } else {
        $form .= zen_href_link($action, '', 'NONSSL');
      }
    }

This looks like a logical error to me. I would expect the first and the third "NONSSL" to be "SSL"...

[DrByte]

When the admin is reworked to handle SSL on all "select" pages, then those parameters will be changed.
As it stands now, the *correct* way to get SSL protection on *all* admin pages is to use an HTTPS address in the HTTP_SERVER setting of your admin configure.php file.

There are known problems with trying to flip back and forth between SSL and NONSSL inside the admin. Those are being addressed in the 2.0 release, and appropriate changes will be made at that time.

Until then, that section of code should *not* be changed.