Fix progress reset response (#1080)

Revert request responses to be strings, not header redirects.
RetroAchievements · Aug 10, 2022 · fd3a660 · fd3a660
1 parent e8307aa
commit fd3a660
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/public/controlpanel.php b/public/controlpanel.php
@@ -156,7 +156,6 @@ function ResetProgressForSelection() {
 
   function onResetComplete(data) {
     if (data.substr(0, 2) !== 'OK') {
-      alert(data);
       //showStatusFailure('Error: ' + data);
       $('#loadingiconreset').attr('src', '<?= asset('Images/tick.png') ?>').delay(750).fadeTo('slow', 0.0);
       return;

diff --git a/public/request/user/reset-achievements.php b/public/request/user/reset-achievements.php
@@ -7,18 +7,15 @@
 $achID = requestInputPost('a', null, 'integer');
 
 if (!authenticateFromCookie($user, $permissions, $userDetails)) {
-    redirect(back() . '?e=error');
     exit;
 }
 
 if (!empty($achID) && resetSingleAchievement($user, $achID)) {
-    redirect(back() . '?reset=ok');
+    echo "OK";
     exit;
 }
 
 if (!empty($gameID) && resetAchievements($user, $gameID) > 0) {
-    redirect(back() . '?reset=ok');
+    echo "OK";
     exit;
 }
-
-redirect(back() . '?e=error');