From 4b3cf019c358b310ec42f7f30ca14faf00e72bac Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Thu, 18 Feb 2021 13:56:15 +0100
Subject: [PATCH] Fixed flickering popup massage preventing approving AFTER
 withdrawing a rejection

This happened because of a conflict between HTML forms native submit behaviour
(automatically refreshing page) and our javascripts submit onclick functions.
Now fixed by setting event.preventDefault() within the js functions.

Signed-off-by: cris-technikum <hainberg@technikum-wien.at>
---
 .../js/lehre/anrechnung/approveAnrechnungDetail.js   | 12 ++++++++++++
 .../lehre/anrechnung/approveAnrechnungUebersicht.js  |  7 +++++++
 public/js/lehre/anrechnung/reviewAnrechnungDetail.js |  3 +++
 .../lehre/anrechnung/reviewAnrechnungUebersicht.js   |  3 +++
 4 files changed, 25 insertions(+)

diff --git a/public/js/lehre/anrechnung/approveAnrechnungDetail.js b/public/js/lehre/anrechnung/approveAnrechnungDetail.js
index d8f0e0874..a01d891b9 100644
--- a/public/js/lehre/anrechnung/approveAnrechnungDetail.js
+++ b/public/js/lehre/anrechnung/approveAnrechnungDetail.js
@@ -43,6 +43,9 @@ $(function(){
     // Approve Anrechnungen
     $("#approveAnrechnungDetail-approve-anrechnung-confirm").click(function(){
 
+        // Avoid form redirecting automatically
+        event.preventDefault();
+
         // Get form data
         let form_data = $('form').serializeArray();
 
@@ -160,6 +163,9 @@ $(function(){
     // Request Recommendation for Anrechnungen
     $("#approveAnrechnungDetail-request-recommendation").click(function(){
 
+        // Avoid form redirecting automatically
+        event.preventDefault();
+
         // Get form data
         let form_data = $('form').serializeArray();
 
@@ -202,6 +208,9 @@ $(function(){
     // Request Recommendation for Anrechnungen
     $("#approveAnrechnungDetail-withdraw-anrechnung-approvement").click(function(){
 
+        // Avoid form redirecting automatically
+        event.preventDefault();
+
         if(!confirm(FHC_PhrasesLib.t("anrechnung", "genehmigungAblehnungWirklichZuruecknehmen")))
         {
             return;
@@ -221,14 +230,17 @@ $(function(){
             {
                 successCallback: function (data, textStatus, jqXHR)
                 {
+                    console.log(data);
                     if (data.error && data.retval != null)
                     {
+                        console.log('inside error');
                         // Print error message
                         FHC_DialogLib.alertWarning(data.retval);
                     }
 
                     if (!data.error && data.retval != null)
                     {
+                        console.log('inside success');
                         approveAnrechnungDetail.formatGenehmigungIsWithdrawed(
                             data.retval.status_bezeichnung
                         );
diff --git a/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js b/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
index 4ec1e1d9d..b1521c308 100644
--- a/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
+++ b/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
@@ -223,6 +223,9 @@ $(function(){
     // Approve Anrechnungen
     $("#approveAnrechnungUebersicht-approve-anrechnungen-confirm").click(function(){
 
+        // Avoid form redirecting automatically
+        event.preventDefault();
+
         // Get selected rows data
         let selected_data = $('#tableWidgetTabulator').tabulator('getSelectedData')
             .map(function(data){
@@ -362,6 +365,10 @@ $(function(){
 
     // Request Recommendation for Anrechnungen
     $("#approveAnrechnungUebersicht-request-recommendation").click(function(){
+
+        // Avoid form redirecting automatically
+        event.preventDefault();
+
         // Get selected rows data
         let selected_data = $('#tableWidgetTabulator').tabulator('getSelectedData');
 
diff --git a/public/js/lehre/anrechnung/reviewAnrechnungDetail.js b/public/js/lehre/anrechnung/reviewAnrechnungDetail.js
index 705e6c440..fc6ffa0cc 100644
--- a/public/js/lehre/anrechnung/reviewAnrechnungDetail.js
+++ b/public/js/lehre/anrechnung/reviewAnrechnungDetail.js
@@ -49,6 +49,9 @@ $(function(){
     // Recommend Anrechnung
     $("#reviewAnrechnungDetail-recommend-anrechnung-confirm").click(function(){
 
+        // Avoid form redirecting automatically
+        event.preventDefault();
+
         // Get form data
         let form_data = $('form').serializeArray();
 
diff --git a/public/js/lehre/anrechnung/reviewAnrechnungUebersicht.js b/public/js/lehre/anrechnung/reviewAnrechnungUebersicht.js
index f9bbbc310..fcc82804e 100644
--- a/public/js/lehre/anrechnung/reviewAnrechnungUebersicht.js
+++ b/public/js/lehre/anrechnung/reviewAnrechnungUebersicht.js
@@ -221,6 +221,9 @@ $(function(){
     // Recommend Anrechnungen
     $("#reviewAnrechnungUebersicht-recommend-anrechnungen-confirm").click(function(){
 
+        // Avoid form redirecting automatically
+        event.preventDefault();
+
         // Get selected rows data
         let selected_data = $('#tableWidgetTabulator').tabulator('getSelectedData')
             .map(function(data){