Ticket #432 (closed defect: duplicate)

Opened 11 months ago

Last modified 10 months ago

[PATCH] Session table creation fails

Reported by: felix.mcco..@gmail.com Assigned to:
Priority: critical Milestone: 0.9
Component: Plugin: DataMapper Keywords:
Cc:

Description

If using DM sessions but no table defined, the DM plugin attempts to create the table resulting in an error.

MySQL error: ERROR 1170 (42000): BLOB/TEXT column 'session_id' used in key specification without a key length

Attachments

datamapper_session_creation_patch.diff (0.5 kB) - added by mkoukoull..@gmail.com on 01/13/08 06:13:37.
svn diff patch

Change History

(follow-up: ↓ 2 ) 01/11/08 19:17:46 changed by iv..@gweezlebur.com

  • milestone changed from 0.5.0 to 0.5.2.

(in reply to: ↑ 1 ; follow-up: ↓ 3 ) 01/13/08 06:09:39 changed by mkoukoull..@gmail.com

Replying to iv..@gweezlebur.com:

The session_id property should be a string instead of text. One line patch attached for merb_datamapper 0.2.5

(in reply to: ↑ 2 ) 01/13/08 06:12:50 changed by mkoukoull..@gmail.com

Replying to mkoukoull..@gmail.com:

Replying to iv..@gweezlebur.com: The session_id property should be a string instead of text. One line patch attached for merb_datamapper 0.2.5

correction, patch is for merb_datamapper in revision 1293

should the status of this ticket now be changed?

01/13/08 06:13:37 changed by mkoukoull..@gmail.com

  • attachment datamapper_session_creation_patch.diff added.

svn diff patch

01/13/08 06:15:30 changed by mkoukoull..@gmail.com

  • summary changed from Session table creation fails to [PATCH] Session table creation fails.

01/28/08 14:01:20 changed by iv..@gweezlebur.com

  • milestone changed from 0.5.2 to 0.5.3.

01/28/08 21:04:45 changed by iv..@gweezlebur.com

  • milestone changed from 0.5.3 to The Future.

02/05/08 16:59:09 changed by d..@caius.name

  • priority changed from medium to critical.
  • milestone changed from The Future to 0.9.

You can't use sessions with datamapper until this patch is accepted. I wrote up the fix on my blog <http://swedishcampground.com/use-datamapper-sessions-with-merb-datamapper> before I found this ticket & patch. The patch applies what I talk through editing in my blog post, been verified by me and ceej in #datamapper as a working fix.

02/11/08 12:58:40 changed by shayarne..@gmail.com

  • status changed from new to closed.
  • resolution set to duplicate.