Improved access check for client edit.

diff --git a/WEB-INF/templates/footer.tpl b/WEB-INF/templates/footer.tpl
index 09e0d89cf44552bd7259fefcab8cbbde0fc721d9..c5ff71ca36559a3b3cee04cd8096220db839e54f 100644 (file)
--- a/WEB-INF/templates/footer.tpl
+++ b/WEB-INF/templates/footer.tpl
@@ -12,7 +12,7 @@
       <br>
       <table cellspacing="0" cellpadding="4" width="100%" border="0">
         <tr>
       <br>
       <table cellspacing="0" cellpadding="4" width="100%" border="0">
         <tr>

-          <td align="center">&nbsp;Anuko Time Tracker 1.18.29.4579 | Copyright &copy; <a href="https://www.anuko.com/lp/tt_3.htm" target="_blank">Anuko</a> |
+          <td align="center">&nbsp;Anuko Time Tracker 1.18.29.4580 | Copyright &copy; <a href="https://www.anuko.com/lp/tt_3.htm" target="_blank">Anuko</a> |

             <a href="https://www.anuko.com/lp/tt_4.htm" target="_blank">{$i18n.footer.credits}</a> |
             <a href="https://www.anuko.com/lp/tt_5.htm" target="_blank">{$i18n.footer.license}</a> |
             <a href="https://www.anuko.com/lp/tt_7.htm" target="_blank">{$i18n.footer.improve}</a>
             <a href="https://www.anuko.com/lp/tt_4.htm" target="_blank">{$i18n.footer.credits}</a> |
             <a href="https://www.anuko.com/lp/tt_5.htm" target="_blank">{$i18n.footer.license}</a> |
             <a href="https://www.anuko.com/lp/tt_7.htm" target="_blank">{$i18n.footer.improve}</a>

diff --git a/client_edit.php b/client_edit.php
index a1f98249c3be9e013deafd940a2010037203648e..7b572ea7c93f5cf01609c012308ae9ae1e0c3e95 100644 (file)
--- a/client_edit.php
+++ b/client_edit.php
@@ -41,8 +41,13 @@ if (!$user->isPluginEnabled('cl')) {
   header('Location: feature_disabled.php');
   exit();
 }
   header('Location: feature_disabled.php');
   exit();
 }

-
-$cl_id = (int) $request->getParameter('id');
+$cl_id = (int)$request->getParameter('id');
+$client = ttClientHelper::getClient($cl_id, true);
+if (!$client) {
+  header('Location: access_denied.php');
+  exit();
+}
+// End of access checks.

 
 $projects = ttGroupHelper::getActiveProjects();
 
 
 $projects = ttGroupHelper::getActiveProjects();
 


@@ -53,7 +58,6 @@ if ($request->isPost()) {
   $cl_status = $request->getParameter('status');
   $cl_projects = $request->getParameter('projects');
 } else {
   $cl_status = $request->getParameter('status');
   $cl_projects = $request->getParameter('projects');
 } else {

-  $client = ttClientHelper::getClient($cl_id, true);

   $cl_name = $client['name'];
   $cl_address = $client['address'];
   $cl_tax = $client['tax'];
   $cl_name = $client['name'];
   $cl_address = $client['address'];
   $cl_tax = $client['tax'];

diff --git a/mobile/client_edit.php b/mobile/client_edit.php
index b65ac99dc3128209c432f6b0d8a6eab378b9b3d1..ab8686509d1b0b1f24a90679588afcdad7667625 100644 (file)
--- a/mobile/client_edit.php
+++ b/mobile/client_edit.php
@@ -40,8 +40,13 @@ if (!$user->isPluginEnabled('cl')) {
   header('Location: feature_disabled.php');
   exit();
 }
   header('Location: feature_disabled.php');
   exit();
 }

-
-$cl_id = (int) $request->getParameter('id');
+$cl_id = (int)$request->getParameter('id');
+$client = ttClientHelper::getClient($cl_id, true);
+if (!$client) {
+  header('Location: access_denied.php');
+  exit();
+}
+// End of access checks.

 
 $projects = ttGroupHelper::getActiveProjects();
 
 
 $projects = ttGroupHelper::getActiveProjects();
 


@@ -52,7 +57,6 @@ if ($request->isPost()) {
   $cl_status = $request->getParameter('status');
   $cl_projects = $request->getParameter('projects');
 } else {
   $cl_status = $request->getParameter('status');
   $cl_projects = $request->getParameter('projects');
 } else {

-  $client = ttClientHelper::getClient($cl_id, true);

   $cl_name = $client['name'];
   $cl_address = $client['address'];
   $cl_tax = $client['tax'];
   $cl_name = $client['name'];
   $cl_address = $client['address'];
   $cl_tax = $client['tax'];