diff --git a/classes/report_subscriptions_abi.php b/classes/report_subscriptions_abi.php index b8e2bf76..bf78aa76 100644 --- a/classes/report_subscriptions_abi.php +++ b/classes/report_subscriptions_abi.php @@ -21,7 +21,7 @@ class report_subscriptions_abi { function __construct($dbh) { $this->dbh = $dbh; - $this->Debug = 1; + $this->Debug = 0; } function DeleteAllSubscriptions($UserID) { diff --git a/www/report-package-notifications.php b/www/report-package-notifications.php index b84adfb9..b3bbe99e 100644 --- a/www/report-package-notifications.php +++ b/www/report-package-notifications.php @@ -158,29 +158,33 @@ function freshports_Package_Sets() { $num_wl_id = min(sizeof($wl_id_array), 30); $num_ps_id = min(sizeof($package_set_array), 2); - echo '$num_abi_id=' . $num_abi_id . "\n
"; - echo '$num_wl_id=' . $num_wl_id . "\n
"; - echo '$num_ps_id=' . $num_ps_id . "\n
"; - $sql = 'Here we go: '; - echo 'Hitting the for loops
'; + if ($Debug) { + echo '$num_abi_id=' . $num_abi_id . "\n
"; + echo '$num_wl_id=' . $num_wl_id . "\n
"; + echo '$num_ps_id=' . $num_ps_id . "\n
"; + $sql = 'Here we go: '; + echo 'Hitting the for loops
'; + } for ($abi_ix = 0; $abi_ix < $num_abi_id; $abi_ix++) { - echo '$abi_ix=' . $abi_ix . "\n
"; + if ($Debug) echo '$abi_ix=' . $abi_ix . "\n
"; for ($wl_ix = 0; $wl_ix < $num_wl_id; $wl_ix++) { - echo '$wl_ix=' . $wl_ix . "\n
"; + if ($Debug) echo '$wl_ix=' . $wl_ix . "\n
"; for ($ps_ix = 0; $ps_ix < $num_ps_id; $ps_ix++) { - echo '$ps_ix=' . $ps_ix . "\n
"; - echo $User->id . ', ' . $wl_id_array[$wl_ix] . ', ', $abi_id_array[$abi_ix] . ', ' . $package_set_array[$ps_ix]; - # sql was created during development, but never used - $sql .= "$User->id, $wl_id_array[$wl_ix], $abi_id_array[$abi_ix], $package_set_array[$ps_ix]
\n"; - - # instead, we save - $rsa->Save($User->id, $wl_id_array[$wl_ix], $abi_id_array[$abi_ix], $package_set_array[$ps_ix]); - } + if ($Debug) { + echo '$ps_ix=' . $ps_ix . "\n
"; + echo $User->id . ', ' . $wl_id_array[$wl_ix] . ', ', $abi_id_array[$abi_ix] . ', ' . $package_set_array[$ps_ix]; + } + # sql was created during development, but never used + $sql .= "$User->id, $wl_id_array[$wl_ix], $abi_id_array[$abi_ix], $package_set_array[$ps_ix]
\n"; + + # instead, we save + $rsa->Save($User->id, $wl_id_array[$wl_ix], $abi_id_array[$abi_ix], $package_set_array[$ps_ix]); + } } } - echo "

\n\nThis is the SQL
$sql
"; + if ($Debug) echo "

\n\nThis is the SQL
$sql
"; } if ($delete) { @@ -207,11 +211,11 @@ function freshports_Package_Sets() { $abi_id = intval($key_values[1]); $package_set = $key_values[2]; - echo "deleting $watch_list_id $abi_id $package_set
\n"; + if ($Debug) echo "deleting $watch_list_id $abi_id $package_set
\n"; $num_rows = $rsa->Delete($User->id, $watch_list_id, $abi_id, $package_set); if ($num_rows >= 1) { - syslog(LOG_ERR, "The key-value pair deleted more than one row ($num_rows): $watch_list_id ::: $abi_id."); - exit("The key-value pair deleted more than one row ($num_rows): " . $watch_list_id . MY_DELIMITER . $abi_id . MY_DELIMITER . $package_set . " - Not sure what's going on there, but I'm stopping right now."); + syslog(LOG_ERR, "The key-value pair deleted more than one row ($num_rows): $watch_list_id ::: $abi_id."); + exit("The key-value pair deleted more than one row ($num_rows): " . $watch_list_id . MY_DELIMITER . $abi_id . MY_DELIMITER . $package_set . " - Not sure what's going on there, but I'm stopping right now."); } } } @@ -265,30 +269,34 @@ function freshports_Package_Sets() { Choose watch lists:
'; echo freshports_WatchListDDLB($db, $User->id, $wlid, 15, 1); ?> Choose your ABI values:
'; echo freshports_ABI_list($db); ?> + echo '
'; + echo freshports_Package_Sets(); + ?> - Add Selected>>> +
+

- Delete Selected >>> + - list of chosen ABI
Your selected watch list / ABI combinations:
'; echo freshports_ABI_list_watching($User->id, $db); ?>