deon/phpldapadmin
1
0
Fork 0
Code Issues 36 Pull Requests Actions Packages Projects Releases Wiki Activity

Fix to getDirty() when using MD5Updates Trait on attributes

Browse Source
This commit is contained in:
Deon George 2025-01-16 10:33:45 +11:00
parent 6e06caa83b
commit 960e0de5c8
4 changed files with 4 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/Classes/LDAP/Attribute.php
View File

@ -233,10 +233,8 @@ class Attribute implements \Countable, \ArrayAccess
*/
public function isDirty(): bool
{
if ($this->oldValues->count() !== $this->values->count())
return TRUE;
return $this->values->diff($this->oldValues)->count() !== 0;
return ($this->oldValues->count() !== $this->values->count())
|| ($this->values->diff($this->oldValues)->count() !== 0);
}
public function oldValues(array $array): void

7
app/Classes/LDAP/Attribute/Binary/JpegPhoto.php
View File

@ -14,13 +14,6 @@ final class JpegPhoto extends Binary
{
use MD5Updates;
public function __construct(string $name,array $values)
{
parent::__construct($name,$values);
$this->internal = FALSE;
}
public function render(bool $edit=FALSE,bool $old=FALSE,bool $new=FALSE): View
{
return view('components.attribute.binary.jpegphoto')

12
app/Ldap/Entry.php
View File

@ -57,16 +57,8 @@ class Entry extends Model
{
$key = $this->normalizeAttributeKey($key);
if ((! array_key_exists($key,$this->original)) && (! $this->objects->has($key)))
return TRUE;
$current = $this->attributes[$key];
$original = $this->objects->get($key)->values;
if ($current === $original)
return TRUE;
return ! $this->getObject($key)->isDirty();
return ((! array_key_exists($key,$this->original)) && (! $this->objects->has($key)))
|| (! $this->getObject($key)->isDirty());
}
public static function query(bool $noattrs=false): Builder

3
app/Traits/MD5Updates.php
View File

@ -11,9 +11,6 @@ trait MD5Updates
{
public function isDirty(): bool
{
if (! parent::isDirty())
return TRUE;
foreach ($this->values->diff($this->oldValues) as $key => $value)
if (md5(Arr::get($this->oldValues,$key)) !== $value)
return TRUE;