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.

Comments

ericdes’s picture

ericdes commented
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: Unassigned » 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!

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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