FHEM/mod-NUKI
1
0
Fork 0
Code Issues Pull Requests Releases 4 Wiki Activity

Refactor command handler return logic for clarity #41

Merged
marko merged 1 commits from patch-new-sub-with-return into dev 2025-10-16 17:57:17 +00:00
Conversation 0 Commits 1 Files Changed 2 +16 -10
marko commented 2025-10-16 17:57:11 +00:00
Owner
Copy Link

Improved the command handler functions within the Nuki Bridge module
by standardizing return statements for consistency. Each command now
immediately returns after assigning its respective endpoint, ensuring
clearer control flow. This change simplifies the readability of the
code and ensures that all command handlers consistently exit without
proceeding further when their conditions are met.

Additionally, unnecessary checks for the endpoint were removed,
streamlining the handling of responses. These adjustments should help
reduce confusion and make future modifications easier, without
introducing any breaking changes.

Improved the command handler functions within the Nuki Bridge module by standardizing return statements for consistency. Each command now immediately returns after assigning its respective endpoint, ensuring clearer control flow. This change simplifies the readability of the code and ensures that all command handlers consistently exit without proceeding further when their conditions are met. Additionally, unnecessary checks for the endpoint were removed, streamlining the handling of responses. These adjustments should help reduce confusion and make future modifications easier, without introducing any breaking changes.
marko added 1 commit 2025-10-16 17:57:12 +00:00
Refactor command handler return logic for clarity 728319d81e 
Improved the command handler functions within the Nuki Bridge module
by standardizing return statements for consistency. Each command now
immediately returns after assigning its respective endpoint, ensuring
clearer control flow. This change simplifies the readability of the
code and ensures that all command handlers consistently exit without
proceeding further when their conditions are met.

Additionally, unnecessary checks for the endpoint were removed,
streamlining the handling of responses. These adjustments should help
reduce confusion and make future modifications easier, without
introducing any breaking changes.
marko merged commit 728319d81e into dev 2025-10-16 17:57:17 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
marko
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: FHEM/mod-NUKI#41
No description provided.
Delete Branch "patch-new-sub-with-return"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?