Skip to content

Allow Variables type in isValid()#31

Merged
ralflang merged 1 commit into
horde:FRAMEWORK_6_0from
horde6:fix_isvalid
Jun 17, 2026
Merged

Allow Variables type in isValid()#31
ralflang merged 1 commit into
horde:FRAMEWORK_6_0from
horde6:fix_isvalid

Conversation

@amulet1

@amulet1 amulet1 commented Jun 16, 2026

Copy link
Copy Markdown
Member

No description provided.

@amulet1 amulet1 requested a review from ralflang June 16, 2026 17:06
@amulet1 amulet1 marked this pull request as draft June 16, 2026 17:22
@amulet1 amulet1 closed this Jun 16, 2026
@amulet1 amulet1 reopened this Jun 16, 2026
@amulet1 amulet1 marked this pull request as ready for review June 16, 2026 18:03
@amulet1 amulet1 requested a review from TDannhauer June 16, 2026 22:04
public $newFolderSet;

public function isValid($vars, $value): bool
public function isValid(Horde_Variables|Variables $vars, $value): bool

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not necessary

/**
*/
public function isValid(Horde_Variables $vars, $value): bool
public function isValid(Horde_Variables|Variables $vars, $value): bool

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@ralflang ralflang merged commit 7d148f2 into horde:FRAMEWORK_6_0 Jun 17, 2026
0 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants