Skip to content

[3.0] Change return mode to enum - #9859

Open
jdarwood007 wants to merge 1 commit into
SimpleMachines:release-3.0from
jdarwood007:3.0/returnMode
Open

jdarwood007 wants to merge 1 commit into
SimpleMachines:release-3.0from
jdarwood007:3.0/returnMode

Conversation

@jdarwood007

Copy link
Copy Markdown
Member

No description provided.

Comment thread Sources/Db/ReturnMode.php Outdated

case Off = 0;
case SingleRow = 1;
case MultieRow = 2;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Small inconsistency here: the docblock for insert() in DatabaseApiInterface refers to
ReturnMode::MultiRow, but the case itself is named MultieRow. Looks like a typo in the case name.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

At least I spelled it wrong everywhere. Fix sent up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants