as title says.

small errors are riddled throughout the code (missing ')" ";" and the like).

C

Comments

Magnity’s picture

Magnity commented
Status: Active » Fixed

Thanks for pointing these out - the -dev release now installs again.

Magnity’s picture

Magnity commented

[Correction - -dev release will install again once the d.o. packaging script runs again (midnight) - but the changes have been committed.]

socialnicheguru’s picture

socialnicheguru commented
Status: Fixed » Active

Parse error: syntax error, unexpected '{', expecting ')' in /Users/aekanem/Documents/aoweb/drupal/sites/all/modules/storm/storminvoice/storminvoice.install on line 68

remove "{" and "}" and replace with " ' "
missing end " ) "
if (!db_table_exists('storminvoiceitem')) { //if Storminvoiceitem was not active

can't update
The update process was aborted prematurely while running update #6011 in storminvoice.module. All errors have been logged. You may need to check the watchdog database table manually.

Unknown column 'vat' in 'storminvoiceitem' query: ALTER TABLE storminvoiceitem CHANGE vat `tax` FLOAT DEFAULT NULL

Magnity’s picture

Magnity commented

Are you sure that you are running the latest -dev version? The parse errors have been fixed already.

I have just committed a fix to the other issue - it now checks for the column existing prior to changing rather than looking at the last update of the invoiceitem module (http://drupal.org/cvs?commit=228772).

Magnity’s picture

Magnity commented

Can you confirm whether these errors are still present?

Thanks,

Magnity’s picture

Magnity commented
Status: Active » Fixed

Marking as fixed as others now seem to be able to install / update fine.

Please reopen if you are stil having issues.

socialnicheguru’s picture

socialnicheguru commented

seems fine to me :) thanks!

Status: Fixed » Closed (fixed)

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