[dokuwiki] New PR: authpdo: Add multi-rowset support and fix mybb test

  • From: "phy25" <wiki@xxxxxxxxxxxx>
  • To: dokuwiki@xxxxxxxxxxxxx
  • Date: Mon, 9 Sep 2019 08:58:02 +0200 (CEST)

Hi,

phy25 opened a new pull request at 
https://github.com/splitbrain/dokuwiki/pull/2850:

- authpdo: add support for multi-rowset to _query
  This makes sure if user defined multiple query in one SQL, they are all 
executed rather than only the first one.
- authpdo: fix mybb schema for MySQL 5.7+
  https://dev.mysql.com/doc/refman/5.7/en/upgrading-from-previous-series.html
  > In MySQL 5.7.5, these SQL mode changes were made: ... Strict SQL mode for 
transactional storage engines (`STRICT_TRANS_TABLES`) is now enabled by default.
- lock fetch PHPUnit version to 7 for PHP 5

One thing I am not sure is whether it's safe to use `nextRowset()` since I read 
about various support about this feature which make it sometimes broken.

Fixes #2841.

Remaining task is to adapt to nightly PHP 8.

Please help us to review this pull request, so new contributors get feedback in 
a timely manner.

29f472a0-d2cf-11e9-8008-dfd01dc5ba79


-- 
DokuWiki mailing list - more info at
http://www.dokuwiki.org/mailinglist

Other related posts:

  • » [dokuwiki] New PR: authpdo: Add multi-rowset support and fix mybb test - phy25