Closed (fixed)
Project:
Asynchronous Prefetch Database Query Cache
Version:
7.x-1.0-beta4
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Oct 2015 at 17:14 UTC
Updated:
9 Nov 2015 at 16:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
dasginganinjaIt appears that line 813 is this line:
$result = apdqc_query(array('system'), array(), "SELECT * FROM " . apdqc_fast_prefix_tables('{' . apdqc_fast_escape_table('system') . "} WHERE name = 'system'") . " LIMIT 1", array('log' => FALSE));. The conditional above that saysif (empty($requirements))so that would lead me to believe that I have no requirements left to do.The server doesn't have the mysqlnd extension just for starters.
Comment #4
mikeytown2 commentedThanks for the bug report! I've committed this patch, let me know if you're still getting this error.
Comment #5
dasginganinjaI'm no longer getting that error I reported earlier.
The error I have now is:
Is this the expected outcome when php-mysqlnd isn't installed?
Comment #7
mikeytown2 commentedThat is not the expected outcome; I've updated the install file with this patch. Let me know how things work out with this latest change.
Comment #8
mikeytown2 commentedAlso did this small patch for windows
Comment #10
dasginganinjaOne thing to note from #5:
I was able to install after I made the changes to settings.php
Now:
Downloaded the latest dev (e12bb2f) (switched to git)
Replaced database with backup before apqdc install
Went to install:
Down the rabbit hole we go. :)
Comment #11
dasginganinjaI broke it out a little bit into this:
and the
!empty(PHP_BINARY)is where the error is at.Comment #12
dasginganinjaWould it make more sense to use
to see if mysqlnd is enabled?
That way it's going to return what is currently being used (in the event that it differs between cli / fpm et al.
Comment #14
mikeytown2 commentedThanks for the bug report; committed this patch. Just a heads up that the problematic code works great on both of my OSes (windows & centos).
Comment #15
mikeytown2 commentedGreat idea; not sure why I didn't think of that sooner! This bit of code has been causing issues for a long time: #2544964-16: Crashing MySQL 5.6.23
Comment #17
mikeytown2 commentedWent this route; one less thing I need to worry about now :)
Comment #18
dasginganinjaI was just uploading a patch that I just wrote and was notified that you did it before I did. AHH
Here's the results on testing 1a49c45
Would it be possible to list the requirements in the error or is that getting a little too helpful? :)I see that there should be some requirements shown based on the code. :)
Comment #19
dasginganinjaJust throwing this out there I guess there's also an
function.
Comment #21
mikeytown2 commentedCommitted this change. The old way of using exec() wasn't reliable thus I didn't throw an error if the array was empty. I think I have this covered now so I made the check better.
Comment #22
jaykainthola commentedI have uploaded latest dev module into server and when trying to install module, I am getting e
I have checked settings.php and there is correct database configuration settings and also added configuration which is suggested for module.
Can you suggest me solution for same?
Thanks
Comment #23
dasginganinjamikeytown2,
I just grabbed the latest release from git (91788efe126fe84e7650de25fce14fc54c1c73f1) and the output is still this:
I'm guessing that isn't the expected output according to what I'm seeing in the code.
Comment #24
mikeytown2 commentedRemoved the fallback detection. Should give the correct warning now.
Comment #26
dasginganinjaNo dice on the last commit.
The issue is at Line 130 where it is checking
$phaseif its value is'runtime'. Shouldn't it be checking for'install'or should the mysqlnd check be placed outside of this conditional?Hopefully this helps with the debugging.
Comment #28
mikeytown2 commentedThanks for pointing this out! Patch has been committed.
Comment #29
dasginganinjaI'd say that the output looks good.
Cheers!