Merge pull request #142 from geodes-sms/feature-129

fixed "Force inclusion criteria selection" issue #129
geodes-sms · May 9, 2024 · 9bba298 · 9bba298
2 parents 562be74 + da2886e
commit 9bba298
Showing 1 changed file with 5 additions and 12 deletions.
diff --git a/relis_app/views/screening/screen_paper.php b/relis_app/views/screening/screen_paper.php
@@ -203,23 +203,16 @@
                     <script>
 
                         function validate_screen() {
-
-                            if ($('.exclusion_crit').css('display') != 'none') {
-                                var crit = $('#criteria_ex').val();
-
-                                if (crit == '0' || crit == '') {
+                            if ($('#decision').val() == 'excluded' && $('#criteria_ex').val() == '') {
                                     alert("You must select an exclusion criteria");
                                     return false;
-                                } else {
-                                    return true;
+                                } else if ($('#decision').val()=='accepted' && $('#criteria_in').val() == '') {                                    
+                                    alert("You must select an inclusion criteria");
+                                    return false;
                                 }
-                            } else {
                                 return true;
                             }
 
-
-                        }
-
                         function include_paper() {
 
                             var content = $('.screen_decision_include').html();
@@ -257,4 +250,4 @@ function exclude_paper() {
         </div>
     </div>
 </div>
-<!-- /page content -->
+<!-- /page content -->