metaform impl: checkbox opts as bool in php

diff --git a/metaform.php b/metaform.php
index e490f37..dc5173b 100644 (file)
--- a/metaform.php
+++ b/metaform.php
@@ -84,7 +84,7 @@ function tem_set_globals(&$tem) {
                'opt_display',
                'opt_pass');
        foreach($bools as $bool) {
-               if(format_bool($GLOBALS[$bool])) {
+               if($GLOBALS[$bool]) {
                        $tem->set($bool);
                }
        }
@@ -101,11 +101,11 @@ function metaform() {
                }
 
                $GLOBALS['singular'] = format_oneline($_REQUEST['singular']);
-               $GLOBALS['opt_email'] = format_yesno($_REQUEST['opt_email']);
-               $GLOBALS['opt_db'] = format_yesno($_REQUEST['opt_db']);
-               $GLOBALS['opt_listing'] = format_yesno($_REQUEST['opt_listing']);
-               $GLOBALS['opt_display'] = format_yesno($_REQUEST['opt_display']);
-               $GLOBALS['opt_pass'] = format_yesno($_REQUEST['opt_pass']);
+               $GLOBALS['opt_email'] = format_bool($_REQUEST['opt_email']);
+               $GLOBALS['opt_db'] = format_bool($_REQUEST['opt_db']);
+               $GLOBALS['opt_listing'] = format_bool($_REQUEST['opt_listing']);
+               $GLOBALS['opt_display'] = format_bool($_REQUEST['opt_display']);
+               $GLOBALS['opt_pass'] = format_bool($_REQUEST['opt_pass']);
 
                tem_init();
                tem_set_globals($GLOBALS['wfpl_template']);
@@ -307,7 +307,7 @@ function make_html($whole_file = true) {
        }
 
        # Submit/Send button
-       if($GLOBALS['opt_email'] == 'Yes' && $GLOBALS['opt_db'] != 'Yes') {
+       if($GLOBALS['opt_email'] == 'Yes' && !$GLOBALS['opt_db']) {
                $visible_fields[] = array('submit' => array(
                        'name' => 'send',
                        'caption' => 'Send'));
@@ -426,7 +426,7 @@ function make_php() {
                        }
                }
 
-               if($GLOBALS['opt_listing'] == 'Yes') {
+               if($GLOBALS['opt_listing']) {
                        if(show_in_listing($type, $input, $format, $sql)) {
                                $tem->show('listing_fields_1');
                                $tem->show('listing_fields_2');
@@ -447,7 +447,7 @@ function make_php() {
        $tem->set('always_field', $always_field);
        $tem->set('db_fields', $db_fields);
        $tem->set('metaform_url', edit_url());
-       if($GLOBALS['opt_email'] == 'Yes') {
+       if($GLOBALS['opt_email']) {
                $this_domain = $_SERVER['HTTP_HOST'];
                if(substr($this_domain, -2) == '.l') {
                        $this_domain = substr($this_domain, 0, -1) . 'com';
@@ -510,7 +510,7 @@ function preview() {
        tem_set('fields', $_REQUEST['fields']);
        $preview_tem = new tem();
        $preview_tem->load_str(make_html(false));
-       if($GLOBALS['opt_db'] == 'Yes') {
+       if($GLOBALS['opt_db']) {
                $preview_tem->show('new_msg');
        }
        $fields = get_fields();
@@ -538,10 +538,10 @@ function download_tar() {
                "style.css" => read_whole_file('code/wfpl/metaform/style.css'),
                "$name.html" => make_html(),
                "$name.php" => make_php());
-       if($GLOBALS['opt_db'] == 'Yes') {
+       if($GLOBALS['opt_db']) {
                $data["$name.sql"] = make_sql();
        }
-       if($GLOBALS['opt_email'] == 'Yes') {
+       if($GLOBALS['opt_email']) {
                $data["$name.email.txt"] = make_email();
        }
        make_tar($name, $data);

diff --git a/metaform/main.html b/metaform/main.html
index 8cd9248..f168fc5 100644 (file)
--- a/metaform/main.html
+++ b/metaform/main.html
@@ -30,7 +30,9 @@
 
     <p><input type="checkbox" name="opt_display~opt_display checked~" value="Yes"> Provide a "Display" page.</p>
 
-    <p><input type="checkbox" name="opt_pass~opt_pass checked~" value="Yes"> Password protect this form.</p>
+    <p><input type="checkbox" name="opt_pass~opt_pass checked~" value="Yes"> Password protect this form.<br>
+       &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="opt_public_form~opt_public_form checked~" value="Yes"> But with publically accessible entry form<br>
+       &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="opt_public_display~opt_public_display checked~" value="Yes"> But with publically accessible view page.</p>
 
        <p>Below, specify the fields you'd like in your form, one field per line. After each field name, put at least one space, then the field type. The following field types are available: <!--~types {~--><!--~types_sep {~-->, <!--~}~-->~type html~<!--~}~-->.</p>