variable-prefix click2bookmark_content_type_ too long [fixed]

svogel - November 14, 2007 - 20:08
Project:Click2Bookmark
Version:5.x-1.2
Component:Code
Category:bug report
Priority:normal
Assigned:ericdes
Status:closed
Description

I just tried to configure a content-type to feature the "add bookmark" but the settings weren't made.

Until then I realised that my content-type-name: "ecommerce_product_toys" was much too long, so that the variable
"click2bookmark_content_type_ecommerce_product_toys" doesn't fit in the name-column (varchar(48)).

I think my content-type name is a bit verbose and long. But as far as I'm concerned, I'd like to have the prefix "click2bookmark_content_type_" shortend.
And because this is set with a define, this shouldn't be a problem.

Otherwise, thanks for this nice module.

By the way. In contrast to a comment in another issue: The feature to enable bookmarks per content-type is really necessary. I didn't want to confuse users because they are able to bookmark the terms&conditions-page.

#1

ericdes - December 22, 2007 - 11:14
Title:variable-prefix click2bookmark_content_type_ too long» variable-prefix click2bookmark_content_type_ too long [fixed]
Version:5.x-1.x-dev» 5.x-1.2
Assigned to:Anonymous» ericdes
Status:active» fixed

I upgraded to Drupal 5.5 and it seems the 'name' column (in the 'variable' table) has now a length of 128 characters. This fixes your problem and you might just as well change the size of the column in previous releases of Drupal.

Thank you for your feedback!

#2

Anonymous - January 5, 2008 - 11:21
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.