-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix phpDoc for getValue function #1015
Comments
Merged
stormwalkerec
added a commit
to stormwalkerec/php-mysqli-database-class
that referenced
this issue
Mar 31, 2024
Fixes ThingEngineer#1016 by setting the defaults to an empty string; This matches nothing without notices about nulls. Fixes ThingEngineer#1013 by defaulting to false instead of not initializing. Fixes ThingEngineer#1015 by adding the missing null in the docstring. Re-aligned the other lines and fixed a typo. Fixes ThingEngineer#1001 by special handling of stdClass objects
ThingEngineer
added a commit
that referenced
this issue
May 24, 2024
Trace Defaults, Bug Fixes (#1001, #1013, #1015, #1016, #1018) Trace Defaults: (Commits: 295c4c6, 057d820) Establishes default settings for tracing mechanisms within the codebase. Bug Fixes: Fixes an error related to calling the end method on an stdClass object (Commit: d181606 - fixes #1001). Resolves issues reported in tickets #1013, #1015, and #1016. Additional Fix (stormwalkerec/php-mysqli-database-class): (Commit: a8de11f - fixes #1018) Merges a separate fix from stormwalkerec/php-mysqli-database-class that addresses another reported issue (#1018).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Current phpDoc for getValue is this :
But $limit can be null to get an array for a single column, so this should change to :
* @param int|null $limit Limit of rows to select. Use null for unlimited..1 by default
The text was updated successfully, but these errors were encountered: