Install script fails due to misplaced break in switch

apsivam - June 14, 2007 - 04:56
Project:Views Argument API
Version:5.x-1.0
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

I get "Views Argument API tables could not be created." error when activating the module.
Upon inspecting I found you set the break statement before creating second table for mysql and mysqli in the script (line no 16 on views_argument_api.install file). I hope this should come after creating the second table.

#1

apsivam - June 14, 2007 - 05:04
Title:Install script error» patch attached
Status:active» needs review

patch file attached for the above mentioned problem

AttachmentSize
install_fix_0.patch 731 bytes

#2

mennonot - June 19, 2007 - 20:55

I had the same problem with installing the 2007-Jun-11 Views Argument API release and the above patch fixed it for me.

#3

Tiburón - June 24, 2007 - 18:17
Title:patch attached» Install script fails due to misplaced break in switch
Priority:normal» critical

Patch in #1 worked here too so +1.

Bumping priority up to critical as the module will not work out of the box unless this bug is fixed.

#4

GearWorker - June 27, 2007 - 13:42

Tnx that fix worked!

#5

jennycita - June 30, 2007 - 00:59

Thanks for posting the fix! I have one more minor change for postgres users... we need to change the type of column views_argument_api_args.api_values from "longtext" to "text", since there is no longtext in postgres. I've attached another patch file with both fixes.

AttachmentSize
install_fix_1.patch 869 bytes

#6

Morris Singer - July 12, 2007 - 00:46

Thanks. Fixed in 5.x-1.5.

#7

Morris Singer - July 12, 2007 - 00:49
Status:needs review» fixed

Marking fixed.

#8

Anonymous - July 26, 2007 - 01:18
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.