Upgrade to Framework 15

Author: pbchase

WarnUsersOfPendingSuspension.php

@@ -47,9 +47,8 @@ function extend_suspension_time($username='')
 			$message = "Unable to extend account suspension time: you need to log in with your credentials.";
 		}
 		else {
-			$sql = "update redcap_user_information set user_lastlogin = NOW() where username ='$username';";
-
-			db_query($sql);
+			# Update the user's last login time to now
+			$result = $this->query("update redcap_user_information set user_lastlogin = NOW() where username = ?;", [$username]);
 		}
 
         if ($message) {
@@ -95,9 +94,9 @@ function warn_users_account_suspension_cron()
 					from redcap_user_information
 					where user_suspended_time is null
 					) as my_user_info
-					where '$suspend_users_inactive_days' - DATEDIFF(NOW(), user_last_date) = '$day';";
+					where ? - DATEDIFF(NOW(), user_last_date) = ?;";
 
-			$q = $this->query($sql);
+					$q = $this->query($sql, [$suspend_users_inactive_days, $day]);
 
 			while ($row = db_fetch_assoc($q))
 			{

config.json

@@ -1,11 +1,8 @@
 {
     "name": "Warn Users of Pending Suspension",
     "namespace": "UF_CTSI\\WarnUsersOfPendingSuspension",
-    "description": "A REDCap external module that will warn users of pending suspensions and provide an easy opportunity to extend the life of REDCap accounts.",
-
-    "permissions": [
-        "redcap_every_page_top"
-    ],
+    "description": "A REDCap external module that warns users of pending account suspensions and provides a simple way for REDCap users to extend their account lifetimes.",
+    "framework-version": 15,
     "enable-every-page-hooks-on-system-pages": true,
     "authors": [
         {