Merge pull request #5924 from BOINC/dpa_web12

web: fix two undefined var bugs
BOINC · Nov 28, 2024 · 98b89e5 · 98b89e5
2 parents dc3597d + 07ee065
commit 98b89e5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/html/user/forum_forum.php b/html/user/forum_forum.php
@@ -109,7 +109,7 @@ function forum_page($forum, $user, $msg=null) {
         if (is_subscribed(-$forum->id, $subs)) {
             BoincNotify::delete_aux(sprintf(
                 'userid=%d and type=%d and opaque=%d',
-                $logged_in_user->id,
+                $user->id,
                 NOTIFY_SUBSCRIBED_FORUM,
                 $forum->id
             ));

diff --git a/html/user/submit.php b/html/user/submit.php
@@ -284,6 +284,8 @@ function handle_admin($user) {
     if (!$user_submit) error_page('no access');
     page_head("Administer job submission");
     if ($user_submit->manage_all) {
+        // user can administer all apps
+        //
         echo "<li>All applications<br>
             <ul>
             <li> <a href=submit.php?action=admin_all>View all batches</a>
@@ -303,6 +305,9 @@ function handle_admin($user) {
             ";
         }
     } else {
+        // see if user can administer specific apps
+        //
+        $usas = BoincUserSubmitApp::enum("user_id=$user->id");
         foreach ($usas as $usa) {
             $app = BoincApp::lookup_id($usa->app_id);
             echo "<li>$app->user_friendly_name<br>