Skip to content

Mistake in Regexp #623

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
Momenoor opened this issue May 26, 2017 · 2 comments
Closed

Mistake in Regexp #623

Momenoor opened this issue May 26, 2017 · 2 comments

Comments

@Momenoor
Copy link

Momenoor commented May 26, 2017 •
edited
Loading

case "bool":
$regexp = "/^[yes|no|0|1|true|false]$/i";
break;
Located in dbObject Line no.657

it must replace [ ] to be ( ) to work correctly
@avbdr
Copy link
Collaborator

avbdr commented Jun 2, 2017

thanks for a finding. can you please submit a pull request with added testcase into tests/ file?

avbdr added a commit to avbdr/PHP-MySQLi-Database-Class that referenced this issue Aug 8, 2017
@avbdr
Bug ThingEngineer#623: Mistake in Regexp
2481e94
@avbdr
Copy link
Collaborator

avbdr commented Aug 8, 2017

fixed thanks

@avbdr avbdr closed this as completed Aug 8, 2017
avbdr added a commit that referenced this issue Aug 9, 2017
@avbdr
fixes (#673)
d87decd 
* fixed limit variable ordering in the comments

* Expand readme on autoreconnect and multipe connections usage

* update release numbers

* update copyright years

* Bug #623:  Mistake in Regexp

* Fix for #554 Fixing Join usage with 'USING' instead of 'ON'

* Fix empty exception message
ricwein pushed a commit to ricwein/PHP-MySQLi-Database-Class that referenced this issue Aug 29, 2017
@avbdr @ricwein
fixes (ThingEngineer#673)
f1fc9ff 
* fixed limit variable ordering in the comments

* Expand readme on autoreconnect and multipe connections usage

* update release numbers

* update copyright years

* Bug ThingEngineer#623:  Mistake in Regexp

* Fix for ThingEngineer#554 Fixing Join usage with 'USING' instead of 'ON'

* Fix empty exception message
avbdr added a commit that referenced this issue Nov 29, 2017
@avbdr
fixes (#698)
1f45192 
* fixed limit variable ordering in the comments

* Expand readme on autoreconnect and multipe connections usage

* update release numbers

* update copyright years

* Bug #623:  Mistake in Regexp

* Fix for #554 Fixing Join usage with 'USING' instead of 'ON'

* Fix empty exception message

* fix a typo

* Added LIKE case description into readme

* do not reset connection name when transtation is on

* 2.9 release
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@avbdr @Momenoor