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

Dont add hints for internal attributes. Our hints now also returns a collection.

Browse Source
This commit is contained in:
Deon George 2025-04-08 11:04:31 +10:00
parent 66537dcec8
commit 6b2fb8dee4
2 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -226,12 +226,15 @@ class Attribute implements \Countable, \ArrayAccess
/** /**
* Return the hints about this attribute, ie: RDN, Required, etc * Return the hints about this attribute, ie: RDN, Required, etc
* *
* @return array * @return Collection
*/ */
public function hints(): array public function hints(): Collection
{ {
$result = collect(); $result = collect();
if ($this->is_internal)
return $result;
// Is this Attribute an RDN // Is this Attribute an RDN
if ($this->is_rdn) if ($this->is_rdn)
$result->put(__('rdn'),__('This attribute is required for the RDN')); $result->put(__('rdn'),__('This attribute is required for the RDN'));
@ -246,7 +249,7 @@ class Attribute implements \Countable, \ArrayAccess
if ($this->isDynamic()) if ($this->isDynamic())
$result->put(__('dynamic'),__('These are dynamic values present as a result of another attribute')); $result->put(__('dynamic'),__('These are dynamic values present as a result of another attribute'));
return $result->toArray(); return $result;
} }
/** /**

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

@ -24,11 +24,11 @@ final class RDN extends Attribute
}; };
} }
public function hints(): array public function hints(): Collection
{ {
return [ return collect([
'required' => __('RDN is required') 'required' => __('RDN is required')
]; ]);
} }
public function render(bool $edit=FALSE,bool $old=FALSE,bool $new=FALSE): View public function render(bool $edit=FALSE,bool $old=FALSE,bool $new=FALSE): View