Download & Extend
Maxlength » Issues

dashes must be replaced with underscores

Posted by kulfi on September 5, 2008 at 5:05am
Project:Maxlength
Version:5.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Installing this module in a fresh Drupal install works fine. However my Drupal app has +50 modules, and I'm not sure if that is causing a conflict, but the module is failing. Its JS isn't included, and there is no indication that it is enabled and configured (it is) when editing the node.

Any ideas?

Login or register to post comments

Comments

#1

Posted by mariuss on September 5, 2008 at 5:39am
Status:active» postponed (maintainer needs more info)

I am not aware of any conflicts, but that is possible.

The only thing I can suggest is that you disable other modules until maxlength starts working, and then you can see what module it conflicts with.

#2

Posted by a_c_m on September 27, 2008 at 4:19pm
Category:support request» bug report

Does your content type's machine name have a "-" in it?

This seems to cause a problem on my test install. Working on a solution now.

#3

Posted by a_c_m on September 27, 2008 at 4:31pm

Not sure if this is the bug you described. However the "_" in the type issue, can be fixed quite simply.

search and replace

<?php
$type = arg(1);
?>

with
<?php
$type = str_replace('-','_',arg(2));
?>

This is because, from the CCK edit content type page under Type:

The machine-readable name of this content type. This text will be used for constructing the URL of the create content page for this content type. This name may consist of only of lowercase letters, numbers, and underscores. Dashes are not allowed. Underscores will be converted into dashes when constructing the URL of the create content page. This name must be unique to this content type.

#4

Posted by mariuss on September 27, 2008 at 5:48pm
Title:Conflicts?» dashes must be replaced with underscores
Status:postponed (maintainer needs more info)» fixed

@a_c_m, thanks for the fix, just committed to CVS. It should show up in the dev version soon.

@ulfk, can you please test with the new dev version (when it becomes available)? It is just a guess that this was causing your problem. Please let us know.

#5

Posted by Anonymous (not verified) on October 11, 2008 at 5:52pm
Status:fixed» closed (fixed)

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