Skip to content

[FEATURE] Add class for querying by parameters#27

Open
beatrycze-volk wants to merge 6 commits into
slub:masterfrom
beatrycze-volk:param
Open

[FEATURE] Add class for querying by parameters#27
beatrycze-volk wants to merge 6 commits into
slub:masterfrom
beatrycze-volk:param

Conversation

@beatrycze-volk

Copy link
Copy Markdown
Collaborator

No description provided.

@beatrycze-volk beatrycze-volk added this to the PHP MODS Reader 0.5.0 milestone Jun 19, 2026
@beatrycze-volk beatrycze-volk self-assigned this Jun 19, 2026
@beatrycze-volk beatrycze-volk added the enhancement New feature or request label Jun 19, 2026
@codacy-production

codacy-production Bot commented Jun 19, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 16 complexity · 0 duplication

Metric Results
Complexity 16
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Adds scalar type declaration to the `xPath` property and removes a debug `var_dump` statement. Includes minor whitespace adjustments and refactoring for consistency.
@beatrycze-volk beatrycze-volk force-pushed the param branch 2 times, most recently from b510d69 to cb9c302 Compare June 19, 2026 13:23
@beatrycze-volk beatrycze-volk force-pushed the param branch 3 times, most recently from f8fd433 to 5c5b127 Compare June 19, 2026 13:44
Consolidates complex `if/else if` conditions for `$query`, `$attributes`, and `$value` parameters. This improves readability and maintainability by streamlining the logic for building XPath predicates.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant