From 6446e424a9b41d3dc6e13d24dc520eb323e02712 Mon Sep 17 00:00:00 2001
From: Cris <hainberg@technikum-wien.at>
Date: Tue, 23 Feb 2021 11:27:03 +0100
Subject: [PATCH] Fixed: Now all buttons have a type (button / reset)

+ belonging adaptations in JS:
. removed e.preventDefault as no longer needed with button type=button
. removed empty text value settings with button type=reset

Signed-off-by: cris-technikum <hainberg@technikum-wien.at>
---
 .../anrechnung/approveAnrechnungDetail.php    | 13 ++++++++-----
 .../approveAnrechnungUebersicht.php           | 19 ++++++++++++-------
 .../anrechnung/reviewAnrechnungDetail.php     | 10 ++++++----
 .../anrechnung/reviewAnrechnungUebersicht.php | 16 ++++++++++------
 .../anrechnung/approveAnrechnungDetail.js     | 18 +++---------------
 .../anrechnung/approveAnrechnungUebersicht.js | 13 ++-----------
 .../anrechnung/reviewAnrechnungDetail.js      |  8 +-------
 .../anrechnung/reviewAnrechnungUebersicht.js  |  8 +-------
 8 files changed, 43 insertions(+), 62 deletions(-)

diff --git a/application/views/lehre/anrechnung/approveAnrechnungDetail.php b/application/views/lehre/anrechnung/approveAnrechnungDetail.php
index e30fb8da5..f5ab892f0 100644
--- a/application/views/lehre/anrechnung/approveAnrechnungDetail.php
+++ b/application/views/lehre/anrechnung/approveAnrechnungDetail.php
@@ -290,11 +290,11 @@ $this->load->view(
                                                 <!-- Action Button 'Abbrechen'-->
                                                 <div class="pull-right">
                                                     <button id="approveAnrechnungDetail-begruendung-abbrechen"
-                                                            class="btn btn-default btn-w200">
+                                                            class="btn btn-default btn-w200" type="reset">
 														<?php echo ucfirst($this->p->t('ui', 'abbrechen')); ?>
                                                     </button>
                                                     <button id="approveAnrechnungDetail-reject-anrechnung-confirm"
-                                                            class="btn btn-primary btn-w200">
+                                                            class="btn btn-primary btn-w200" type="button">
 														<?php echo ucfirst($this->p->t('ui', 'bestaetigen')); ?>
                                                     </button>
                                                 </div>
@@ -311,11 +311,11 @@ $this->load->view(
                                                 <!-- Action Button 'Abbrechen'-->
                                                 <div class="pull-right">
                                                     <button id="approveAnrechnungDetail-genehmigung-abbrechen"
-                                                            class="btn btn-default btn-w200">
+                                                            class="btn btn-default btn-w200" type="reset">
 														<?php echo ucfirst($this->p->t('ui', 'abbrechen')); ?>
                                                     </button>
                                                     <button id="approveAnrechnungDetail-approve-anrechnung-confirm"
-                                                            class="btn btn-primary btn-w200">
+                                                            class="btn btn-primary btn-w200" type="button">
 														<?php echo ucfirst($this->p->t('ui', 'bestaetigen')); ?>
                                                     </button>
                                                 </div>
@@ -333,14 +333,17 @@ $this->load->view(
                 <div class="pull-right">
                     <button id="approveAnrechnungDetail-withdraw-anrechnung-approvement"
                             class="btn btn-default btn-w200 <?php echo ($anrechnungData->status_kurzbz == 'approved' ||
-                                $anrechnungData->status_kurzbz == 'rejected') ? '' : 'hidden' ?>">
+                                $anrechnungData->status_kurzbz == 'rejected') ? '' : 'hidden' ?>"
+                            type="button">
 	                    <?php echo ucfirst($this->p->t('global', 'zuruecknehmen')); ?>
                     </button>
                     <button id="approveAnrechnungDetail-reject-anrechnung-ask" class="btn btn-danger btn-w200"
+                            type="button"
 						<?php echo $anrechnungData->status_kurzbz == 'inProgressDP' ? '' : 'disabled' ?>>
 						<?php echo ucfirst($this->p->t('global', 'ablehnen')); ?>
                     </button>
                     <button id="approveAnrechnungDetail-approve-anrechnung-ask" class="btn btn-primary btn-w200"
+                            type="button"
 						<?php echo $anrechnungData->status_kurzbz == 'inProgressDP' ? '' : 'disabled' ?>>
 						<?php echo ucfirst($this->p->t('global', 'genehmigen')); ?>
                     </button>
diff --git a/application/views/lehre/anrechnung/approveAnrechnungUebersicht.php b/application/views/lehre/anrechnung/approveAnrechnungUebersicht.php
index 91d821fde..7ec175384 100644
--- a/application/views/lehre/anrechnung/approveAnrechnungUebersicht.php
+++ b/application/views/lehre/anrechnung/approveAnrechnungUebersicht.php
@@ -155,11 +155,12 @@ $this->load->view(
                 <br>
                 <!-- Action Button 'Abbrechen'-->
                 <div class="pull-right">
-                    <button id="approveAnrechnungUebersicht-begruendung-abbrechen" class="btn btn-default btn-w200">
+                    <button id="approveAnrechnungUebersicht-begruendung-abbrechen"
+                            class="btn btn-default btn-w200" type="reset">
 						<?php echo ucfirst($this->p->t('ui', 'abbrechen')); ?>
                     </button>
                     <button id="approveAnrechnungUebersicht-reject-anrechnungen-confirm"
-                            class="btn btn-primary btn-w200">
+                            class="btn btn-primary btn-w200" type="button">
 						<?php echo ucfirst($this->p->t('ui', 'bestaetigen')); ?>
                     </button>
                 </div>
@@ -173,11 +174,12 @@ $this->load->view(
                 <br>
                 <!-- Action Button 'Abbrechen'-->
                 <div class="pull-right">
-                    <button id="approveAnrechnungUebersicht-empfehlung-abbrechen" class="btn btn-default btn-w200">
+                    <button id="approveAnrechnungUebersicht-empfehlung-abbrechen"
+                            class="btn btn-default btn-w200" type="reset">
 						<?php echo ucfirst($this->p->t('ui', 'abbrechen')); ?>
                     </button>
                     <button id="approveAnrechnungUebersicht-approve-anrechnungen-confirm"
-                            class="btn btn-primary btn-w200">
+                            class="btn btn-primary btn-w200" type="button">
 						<?php echo ucfirst($this->p->t('ui', 'bestaetigen')); ?>
                     </button>
                 </div>
@@ -221,11 +223,14 @@ $this->load->view(
             <div class="col-xs-9">
                 <div class="pull-right">
                     <button id="approveAnrechnungUebersicht-request-recommendation"
-                            class="btn btn-default btn-w200 btn-mr50"><?php echo ucfirst($this->p->t('anrechnung', 'empfehlungAnfordern')); ?></button>
+                            class="btn btn-default btn-w200 btn-mr50" type="button">
+                        <?php echo ucfirst($this->p->t('anrechnung', 'empfehlungAnfordern')); ?></button>
                     <button id="approveAnrechnungUebersicht-reject-anrechnungen-ask"
-                            class="btn btn-danger btn-w200"><?php echo ucfirst($this->p->t('global', 'ablehnen')); ?></button>
+                            class="btn btn-danger btn-w200" type="button">
+                        <?php echo ucfirst($this->p->t('global', 'ablehnen')); ?></button>
                     <button id="approveAnrechnungUebersicht-approve-anrechnungen-ask"
-                            class="btn btn-primary btn-w200"><?php echo ucfirst($this->p->t('global', 'genehmigen')); ?></button>
+                            class="btn btn-primary btn-w200" type="button">
+                        <?php echo ucfirst($this->p->t('global', 'genehmigen')); ?></button>
                 </div>
             </div>
         </div>
diff --git a/application/views/lehre/anrechnung/reviewAnrechnungDetail.php b/application/views/lehre/anrechnung/reviewAnrechnungDetail.php
index 455c31e65..8e0fc4b46 100644
--- a/application/views/lehre/anrechnung/reviewAnrechnungDetail.php
+++ b/application/views/lehre/anrechnung/reviewAnrechnungDetail.php
@@ -209,11 +209,11 @@ $this->load->view(
                                                 <!-- Action Button Abbrechen & Bestaetigen-->
                                                 <div class="pull-right">
                                                     <button id="reviewAnrechnungDetail-begruendung-abbrechen"
-                                                            class="btn btn-default btn-w200">
+                                                            class="btn btn-default btn-w200" type="reset">
 														<?php echo ucfirst($this->p->t('ui', 'abbrechen')); ?>
                                                     </button>
                                                     <button id="reviewAnrechnungDetail-dont-recommend-anrechnung-confirm"
-                                                            class="btn btn-primary btn-w200">
+                                                            class="btn btn-primary btn-w200" type="button">
 														<?php echo ucfirst($this->p->t('ui', 'bestaetigen')); ?>
                                                     </button>
                                                 </div>
@@ -229,11 +229,11 @@ $this->load->view(
                                                 <!-- Action Button Abbrechen & Bestaetigen-->
                                                 <div class="pull-right">
                                                     <button id="reviewAnrechnungDetail-empfehlung-abbrechen"
-                                                            class="btn btn-default btn-w200">
+                                                            class="btn btn-default btn-w200" type="reset">
 														<?php echo ucfirst($this->p->t('ui', 'abbrechen')); ?>
                                                     </button>
                                                     <button id="reviewAnrechnungDetail-recommend-anrechnung-confirm"
-                                                            class="btn btn-primary btn-w200">
+                                                            class="btn btn-primary btn-w200" type="button">
 														<?php echo ucfirst($this->p->t('ui', 'bestaetigen')); ?>
                                                     </button>
                                                 </div>
@@ -248,10 +248,12 @@ $this->load->view(
 
                 <div class="pull-right">
                     <button id="reviewAnrechnungDetail-dont-recommend-anrechnung-ask" class="btn btn-danger btn-w200"
+                            type="button"
 						<?php echo is_null($empfehlungData->empfehlung) ? '' : 'disabled' ?>>
 						<?php echo ucfirst($this->p->t('anrechnung', 'nichtEmpfehlen')); ?>
                     </button>
                     <button id="reviewAnrechnungDetail-recommend-anrechnung-ask" class="btn btn-primary btn-w200"
+                            type="button"
 						<?php echo is_null($empfehlungData->empfehlung) ? '' : 'disabled' ?>>
 						<?php echo ucfirst($this->p->t('anrechnung', 'empfehlen')); ?>
                     </button>
diff --git a/application/views/lehre/anrechnung/reviewAnrechnungUebersicht.php b/application/views/lehre/anrechnung/reviewAnrechnungUebersicht.php
index 55c82b4e3..c3ca0754a 100644
--- a/application/views/lehre/anrechnung/reviewAnrechnungUebersicht.php
+++ b/application/views/lehre/anrechnung/reviewAnrechnungUebersicht.php
@@ -154,11 +154,12 @@ $this->load->view(
                 <br>
                 <!-- Action Button Abbrechen & Bestaetigen-->
                 <div class="pull-right">
-                    <button id="reviewAnrechnungUebersicht-begruendung-abbrechen" class="btn btn-default btn-w200">
+                    <button id="reviewAnrechnungUebersicht-begruendung-abbrechen"
+                            class="btn btn-default btn-w200" type="reset">
 						<?php echo ucfirst($this->p->t('ui', 'abbrechen')); ?>
                     </button>
                     <button id="reviewAnrechnungUebersicht-dont-recommend-anrechnungen-confirm"
-                            class="btn btn-primary btn-w200">
+                            class="btn btn-primary btn-w200" type="button">
 						<?php echo ucfirst($this->p->t('ui', 'bestaetigen')); ?>
                     </button>
                 </div>
@@ -172,11 +173,12 @@ $this->load->view(
                 <br>
                 <!-- Action Button 'Abbrechen'-->
                 <div class="pull-right">
-                    <button id="reviewAnrechnungUebersicht-empfehlung-abbrechen" class="btn btn-default btn-w200">
+                    <button id="reviewAnrechnungUebersicht-empfehlung-abbrechen"
+                            class="btn btn-default btn-w200" type="reset">
 						<?php echo ucfirst($this->p->t('ui', 'abbrechen')); ?>
                     </button>
                     <button id="reviewAnrechnungUebersicht-recommend-anrechnungen-confirm"
-                            class="btn btn-primary btn-w200">
+                            class="btn btn-primary btn-w200" type="button">
 						<?php echo ucfirst($this->p->t('ui', 'bestaetigen')); ?>
                     </button>
                 </div>
@@ -222,9 +224,11 @@ $this->load->view(
             <div class="col-xs-9">
                 <div class="pull-right">
                     <button id="reviewAnrechnungUebersicht-dont-recommend-anrechnungen-ask"
-                            class="btn btn-danger btn-w200"><?php echo ucfirst($this->p->t('anrechnung', 'nichtEmpfehlen')); ?></button>
+                            class="btn btn-danger btn-w200" type="button">
+                        <?php echo ucfirst($this->p->t('anrechnung', 'nichtEmpfehlen')); ?></button>
                     <button id="reviewAnrechnungUebersicht-recommend-anrechnungen-ask"
-                            class="btn btn-primary btn-w200"><?php echo ucfirst($this->p->t('anrechnung', 'empfehlen')); ?></button>
+                            class="btn btn-primary btn-w200" type="button">
+                        <?php echo ucfirst($this->p->t('anrechnung', 'empfehlen')); ?></button>
                 </div>
             </div>
         </div>
diff --git a/public/js/lehre/anrechnung/approveAnrechnungDetail.js b/public/js/lehre/anrechnung/approveAnrechnungDetail.js
index 33cce7118..872da890b 100644
--- a/public/js/lehre/anrechnung/approveAnrechnungDetail.js
+++ b/public/js/lehre/anrechnung/approveAnrechnungDetail.js
@@ -43,9 +43,6 @@ $(function(){
     // Approve Anrechnungen
     $("#approveAnrechnungDetail-approve-anrechnung-confirm").click(function(e){
 
-        // Avoid form redirecting automatically
-        e.preventDefault();
-
         // Avoid bubbling click event to sibling break button
         e.stopImmediatePropagation();
 
@@ -107,9 +104,6 @@ $(function(){
     // Reject Anrechnungen
     $("#approveAnrechnungDetail-reject-anrechnung-confirm").click(function(e){
 
-        // Avoid form redirecting automatically
-        e.preventDefault();
-
         // Avoid bubbling click event to sibling break button
         e.stopImmediatePropagation();
 
@@ -167,10 +161,7 @@ $(function(){
     });
 
     // Request Recommendation for Anrechnungen
-    $("#approveAnrechnungDetail-request-recommendation").click(function(e){
-
-        // Avoid form redirecting automatically
-        e.preventDefault();
+    $("#approveAnrechnungDetail-request-recommendation").click(function(){
 
         // Get form data
         let form_data = $('form').serializeArray();
@@ -212,10 +203,7 @@ $(function(){
     });
 
     // Request Recommendation for Anrechnungen
-    $("#approveAnrechnungDetail-withdraw-anrechnung-approvement").click(function(e){
-
-        // Avoid form redirecting automatically
-        e.preventDefault();
+    $("#approveAnrechnungDetail-withdraw-anrechnung-approvement").click(function(){
 
         if(!confirm(FHC_PhrasesLib.t("anrechnung", "genehmigungAblehnungWirklichZuruecknehmen")))
         {
@@ -276,7 +264,7 @@ $(function(){
 
     // Break Begruendung abgeben
     $('#approveAnrechnungDetail-begruendung-abbrechen').click(function(){
-        $('#approveAnrechnungDetail-begruendung').val('');
+
         begruendung_panel.slideUp('slow');
 
     })
diff --git a/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js b/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
index b5be4c3a2..112d48f85 100644
--- a/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
+++ b/public/js/lehre/anrechnung/approveAnrechnungUebersicht.js
@@ -223,9 +223,6 @@ $(function(){
     // Approve Anrechnungen
     $("#approveAnrechnungUebersicht-approve-anrechnungen-confirm").click(function(e){
 
-        // Avoid form redirecting automatically
-        e.preventDefault();
-
         // Avoid bubbling click event to sibling break button
         e.stopImmediatePropagation();
 
@@ -298,9 +295,6 @@ $(function(){
     // Reject Anrechnungen
     $("#approveAnrechnungUebersicht-reject-anrechnungen-confirm").click(function(e){
 
-        // Avoid form redirecting automatically
-        e.preventDefault();
-
         // Avoid bubbling click event to sibling break button
         e.stopImmediatePropagation();
 
@@ -370,10 +364,7 @@ $(function(){
     });
 
     // Request Recommendation for Anrechnungen
-    $("#approveAnrechnungUebersicht-request-recommendation").click(function(e){
-
-        // Avoid form redirecting automatically
-        e.preventDefault();
+    $("#approveAnrechnungUebersicht-request-recommendation").click(function(){
 
         // Get selected rows data
         let selected_data = $('#tableWidgetTabulator').tabulator('getSelectedData');
@@ -444,7 +435,7 @@ $(function(){
 
     // Break Ablehnung abgeben
     $('#approveAnrechnungUebersicht-begruendung-abbrechen').click(function(){
-        $('#approveAnrechnungUebersicht-begruendung').val('');
+
         begruendung_panel.slideUp('slow');
 
     })
diff --git a/public/js/lehre/anrechnung/reviewAnrechnungDetail.js b/public/js/lehre/anrechnung/reviewAnrechnungDetail.js
index 6648792bd..d4f8ec5a7 100644
--- a/public/js/lehre/anrechnung/reviewAnrechnungDetail.js
+++ b/public/js/lehre/anrechnung/reviewAnrechnungDetail.js
@@ -49,9 +49,6 @@ $(function(){
     // Recommend Anrechnung
     $("#reviewAnrechnungDetail-recommend-anrechnung-confirm").click(function(e){
 
-        // Avoid form redirecting automatically
-        e.preventDefault();
-
         // Avoid bubbling click event to sibling break button
         e.stopImmediatePropagation();
 
@@ -113,9 +110,6 @@ $(function(){
     // Dont recommend Anrechnung
     $("#reviewAnrechnungDetail-dont-recommend-anrechnung-confirm").click(function(e){
 
-        // Avoid form redirecting automatically
-        e.preventDefault();
-
         // Avoid bubbling click event to sibling break button
         e.stopImmediatePropagation();
 
@@ -180,7 +174,7 @@ $(function(){
 
     // Break Begruendung abgeben
     $('#reviewAnrechnungDetail-begruendung-abbrechen').click(function(){
-        $('#reviewAnrechnungDetail-begruendung').val('');
+
         begruendung_panel.slideUp('slow');
     })
 
diff --git a/public/js/lehre/anrechnung/reviewAnrechnungUebersicht.js b/public/js/lehre/anrechnung/reviewAnrechnungUebersicht.js
index b4efd2cf7..1785bc6b1 100644
--- a/public/js/lehre/anrechnung/reviewAnrechnungUebersicht.js
+++ b/public/js/lehre/anrechnung/reviewAnrechnungUebersicht.js
@@ -221,9 +221,6 @@ $(function(){
     // Recommend Anrechnungen
     $("#reviewAnrechnungUebersicht-recommend-anrechnungen-confirm").click(function(e){
 
-        // Avoid form redirecting automatically
-        e.preventDefault();
-
         // Avoid bubbling click event to sibling break button
         e.stopImmediatePropagation();
 
@@ -297,9 +294,6 @@ $(function(){
     // Dont recommend Anrechnungen
     $("#reviewAnrechnungUebersicht-dont-recommend-anrechnungen-confirm").click(function(e){
 
-        // Avoid form redirecting automatically
-        e.preventDefault();
-
         // Avoid bubbling click event to sibling break button
         e.stopImmediatePropagation();
 
@@ -376,7 +370,7 @@ $(function(){
 
     // Break Begruendung abgeben
     $('#reviewAnrechnungUebersicht-begruendung-abbrechen').click(function(){
-        $('#reviewAnrechnungUebersicht-begruendung').val('');
+
         begruendung_panel.slideUp('slow');
 
     })