Skip to content
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

Object oriented fetchAll #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Object oriented fetchAll #9

wants to merge 1 commit into from

Conversation

klonfish
Copy link

@klonfish klonfish commented May 2, 2013

The version with sqlite_fetch_all($res, SQLITE_NUM) did not work in my setup:

PHP 5.3.10
DokuWiki Adora Belle
data plugin released at 2013-02-06

That means, that it always printed "Nothing." instead of the actual list. The call to sqlite_fetch_all returned NULL.

Replacing that one line by the object oriented style $res->fetchAll(SQLITE_NUM) made it work for me.

Hope it doesn't break other installations.

@Klap-in
Copy link
Contributor

Klap-in commented May 13, 2013

I added another PR #10 which uses the Sqlite plugin. This plugin contains switching between sqlite 2 and sqlite3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants