maket/MaketRandevu
1
1
Fork 0
Code Issues 15 Pull requests Projects Releases 4 Packages Wiki Activity

The set setting method could also get a NULL value

Browse source
This commit is contained in:
Alex Tselegidis 2021-11-05 08:28:59 +01:00
parent bcdd9a28a2
commit 3c5be0d12b
3 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
application/models/Admins_model.php
View file

@ -438,9 +438,9 @@ class Admins_model extends EA_Model {
* *
* @param int $admin_id Admin ID. * @param int $admin_id Admin ID.
* @param string $name Setting name. * @param string $name Setting name.
* @param string $value Setting value. * @param string|null $value Setting value.
*/ */
public function set_setting(int $admin_id, string $name, string $value) public function set_setting(int $admin_id, string $name, string $value = NULL)
{ {
if ( ! $this->db->update('user_settings', [$name => $value], ['id_users' => $admin_id])) if ( ! $this->db->update('user_settings', [$name => $value], ['id_users' => $admin_id]))
{ {

4
application/models/Providers_model.php
View file

@ -464,9 +464,9 @@ class Providers_model extends EA_Model {
* *
* @param int $provider_id Provider ID. * @param int $provider_id Provider ID.
* @param string $name Setting name. * @param string $name Setting name.
* @param string $value Setting value. * @param string|null $value Setting value.
*/ */
public function set_setting(int $provider_id, string $name, string $value) public function set_setting(int $provider_id, string $name, string $value = NULL)
{ {
if ( ! $this->db->update('user_settings', [$name => $value], ['id_users' => $provider_id])) if ( ! $this->db->update('user_settings', [$name => $value], ['id_users' => $provider_id]))
{ {

8
application/models/Secretaries_model.php
View file

@ -280,7 +280,7 @@ class Secretaries_model extends EA_Model {
{ {
if ( ! $this->db->get_where('users', ['id' => $secretary_id])->num_rows()) if ( ! $this->db->get_where('users', ['id' => $secretary_id])->num_rows())
{ {
throw new InvalidArgumentException('The provided secretary ID was not found in the database: ' . $provider_id); throw new InvalidArgumentException('The provided secretary ID was not found in the database: ' . $secretary_id);
} }
$secretary = $this->db->get_where('users', ['id' => $secretary_id])->row_array(); $secretary = $this->db->get_where('users', ['id' => $secretary_id])->row_array();
@ -332,7 +332,7 @@ class Secretaries_model extends EA_Model {
if ( ! $query->num_rows()) if ( ! $query->num_rows())
{ {
throw new InvalidArgumentException('The provided secretary ID was not found in the database: ' . $provider_id); throw new InvalidArgumentException('The provided secretary ID was not found in the database: ' . $secretary_id);
} }
// Check if the required field is part of the secretary data. // Check if the required field is part of the secretary data.
@ -446,9 +446,9 @@ class Secretaries_model extends EA_Model {
* *
* @param int $secretary_id Secretary ID. * @param int $secretary_id Secretary ID.
* @param string $name Setting name. * @param string $name Setting name.
* @param string $value Setting value. * @param string|null $value Setting value.
*/ */
public function set_setting(int $secretary_id, string $name, string $value) public function set_setting(int $secretary_id, string $name, string $value = NULL)
{ {
if ( ! $this->db->update('user_settings', [$name => $value], ['id_users' => $secretary_id])) if ( ! $this->db->update('user_settings', [$name => $value], ['id_users' => $secretary_id]))
{ {