Showstopper in Hotpot CVS

Showstopper in Hotpot CVS

by Dennis Daniels -
Number of replies: 6
Parse error: parse error, unexpected T_SL in /home/dgdaniel/public_html/mod/hotpot/version.php on line 6

Just upped the CVS and I got the error above...

You can see it here:
http://dennisgdaniels.thinkingdistance.org/


Here's the bug report:
http://moodle.org/bugs/bug.php?op=show&bugid=5265&pos=1

Fixes? Suggestions?! smile

Dennis



Average of ratings: -
In reply to Dennis Daniels

Re: Showstopper in Hotpot CVS

by Tim Allen -

I'm getting a different error after a CVS update:

 

hotpot module needs upgrading

(mysql): ALTER TABLE `mdl_hotpot_questions` CHANGE `name` `name` TEXT NOT NULL DEFAULT ''  

1170: BLOB/TEXT column 'name' used in key specification without a key length

               
            
         
      
   

Error

And then:

Upgrading hotpot from 2005090715 to 2006042102 FAILED
And then the site upgrade completely stops!  surprise
In reply to Tim Allen

Re: Showstopper in Hotpot CVS

by Gordon Bateson -
Picture of Core developers Picture of Peer reviewers Picture of Plugin developers
Hi Tim,
thanks for the heads up.

I have committed a fix to the CVS.

Sometimes it can take a day or so for changes to be available to everyone, so if you need the patch now, please replace "mod/hotpot/db/update_to_v2.php" in your Moodle site with the script attached to this message.

cheers
Gordon
In reply to Gordon Bateson

Re: Showstopper in Hotpot CVS

by Tim Allen -
Hi Gordon,

Thanks for posting this patch.  As my other reply shows, I already fixed the problem before I saw your reply by just deleting the database tables and reinstalling, as I didn't have any data in them - but I am sure that this will help others. smile

Cheers,
Tim.
In reply to Tim Allen

Re: Showstopper in Hotpot CVS

by Gordon Bateson -
Picture of Core developers Picture of Peer reviewers Picture of Plugin developers
Hi Tim,
I'm glad to hear you're up and running again smile

If anyone else had this problem, and was later able to complete the upgrade using either the patch supplied earlier or the most recent CVS, please let us know.

all the best
Gordon
In reply to Tim Allen

Re: Showstopper in Hotpot CVS

by Tim Allen -
I took the easy way out and just deleted the database tables and then reinstalled it and it worked. smile

Tim.