Updating to 6.x-1.13 produces the following:

    * warning: array_merge() [function.array-merge]: Argument #2 is not an array in /home/xxxxx/public_html/update.php on line 173.
    * warning: Invalid argument supplied for foreach() in /home/xxxxx/public_html/update.php on line 337.

Update 6008 from print.install should return an array

function print_update_6008() {
  variable_del('print_text_by');
  variable_del('print_text_created');
}

Comments

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Status: Active » Fixed

Hi, thanks for the notification.. Is the status report still wanting you to run update.php for the print module?

Asking this to know if I need to issue 6.x-1.14 because of this or not..

Anyway, fixed in git..

DrewMathers’s picture

DrewMathers CreditAttribution: DrewMathers commented

Status Report is showing Print is up-to-date at 6.x-1.13. Update.php shows now updates pending for Print.

creeksideplayers’s picture

creeksideplayers CreditAttribution: creeksideplayers commented

I'm seeing the same error after updated from 6.x-1.12 to 6.x-1.13.

katetwu’s picture

katetwu CreditAttribution: katetwu commented

I also received the above mentioned error, but can't get to the status report. The following error is what I get trying to get to the status report:

Fatal error: Call to undefined method TCPDF::getTCPDFVersion() in /usr/home/sites/xxxxxxxxx/www/sites/all/modules/print/print_pdf/print_pdf.module on line 613

DrewMathers’s picture

DrewMathers CreditAttribution: DrewMathers commented

katetwu,

Try disabling the Print PDF module. Are you sure you downloaded and installed the TCPDF library correctly?

katetwu’s picture

katetwu CreditAttribution: katetwu commented

Hi Drew,

I tried disabling the Print PDF module and same result. Is the TCPDF library different from the one used by 6.x-1.12? 6.x-1.12 never had a problem.

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Status: Fixed » Active

katetwu, the module now requires TCPDF version 5.9.012 or later.. Please update..

However, the fatal error is important.. The call to getTCPDFVersion should not result in a fatal error if it is not there..

I'll release a 6.x-1,14 version soon because of the missing return array.

DrewMathers’s picture

DrewMathers CreditAttribution: DrewMathers commented

jcnventura,

Before you release a 6.x-1.4, how about putting in a function_exists() check for getTCPDFVersion, so it doesn't blow up if the library is missing/wrong place/wrong capitalization? I know this should be a new issue, but I wanted to catch you before the 6.x-1.4 release.

katetwu’s picture

katetwu CreditAttribution: katetwu commented

Re: TCPDF version 5.9.012 The only download I can find is TCPDF version 5.9.111. Will this version work? I believe it is the latest.

Update: Finally got it to work with tcpdf version 5.9.111 and moving it into the print folder. I don't know if it should be done that way but it worked and status report and cron run correctly.

venusrising’s picture

venusrising CreditAttribution: venusrising commented

Same error here. Also causes access and cron run issues. With pdf enabled still errors out. Breaks cron run. Trying to roll back to previous build does not work. Again breaks stuff and stops cron run from being run manually etc. Not ready for production.

venusrising’s picture

venusrising CreditAttribution: venusrising commented
Priority: Minor » Major

Can no longer run cron via admin menu. Cron fails everytime. Disabled and uninstalled module but cron error still persists.

venusrising’s picture

venusrising CreditAttribution: venusrising commented

Deleted cron last variable and cron will run again. Had to roll back build for the time being until it is fixed.

b0b’s picture

b0b CreditAttribution: b0b commented

subscribe

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Status: Active » Fixed

Fixed.. Rolling 6.x-1.14 now.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

webservant316’s picture

webservant316 CreditAttribution: webservant316 commented
Version: 6.x-1.13 » 6.x-1.18
Status: Closed (fixed) » Active

just got this error

warning: array_merge() [function.array-merge]: Argument #2 is not an array in update.php on line 173.
warning: Invalid argument supplied for foreach() in update.php on line 337.

when updating to print 6.x-1.18 along with other module updates. dunno if it was print again or one of the other modules. any insight?

Is this problem back?

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Status: Active » Postponed (maintainer needs more info)

Try to disable the print module before running update.php.. If it succeeds with all others, and breaks only with print, then it must be the print module. There's a few possibilities that might be creating some errors in the update:

#1778204: update problem because of undefined json_decode
#1774298: Declaration of PrintTCPDF::SetFooterData() should be compatible with that of TCPDF::setFooterData

webservant316’s picture

webservant316 CreditAttribution: webservant316 commented

The problem was upgrading 'flag' from 1.13 to 2.0. Please ignore #16 above. Sorry for the trouble.

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Version: 6.x-1.18 » 6.x-1.13
Status: Postponed (maintainer needs more info) » Closed (fixed)

np