DROP: when the user adds a content type, base the type on the name using javascript

just a question: why does the user even need to enter the machine readable type?

This is assigned to version 7.x-dev, but as a DROP task would we want it implemented in 6.x or 5.x? Should it be a patch in core or should it be a module like "AutoSuggest node-name?"

@snufkin : for non-occidental languages, you can't always autogenerate a suitable machine name (a-z, _) from the human name.

@yched: thats true I havent thought about this.

It just appeared to me because recently i was introducing Drupal to someone completely new to the CMS, and it seemed completely useless and confusing to give a machine name too.

@greggles: is it a necessary control though? In CCK IMHO all of the controls are important and change behaviour some way, this doesnt really.

And actually if I take what yched said, then is it not possible to do this mapping via JS? Or reversed: if its possible to do it with JavaScript, it should be possible to do it via PHP, automatically.

A DROP task certainly can be for 7.x, I just want it NOW :). I think this is awesome as an idea, and if there were a contrib module for it, I'd use it on all my sites (existing 5x and new 6x sites).

I claim this task!!

Here's the patch. Many thanks to Demitri for all his help!! A beer for him as soon as he's 21! :D

Eclipse's .project file got added to this patch by mistake.

ok, took out the project file from the patch

Marking back to needs review. :)

It doesn't work in Firefox 2, this is the error output by Firebug:

$("#edit-name", context).val().strtolower is not a function
http://localhost/drupal09/modules/node/node.admin.js?E
Line 3

The .strtolower method was changed to .toLowerCase This should now work.

It works now.

However, if you change the MRN (machine-readable name) to something else, it will change back as soon as the HRN loses focus again.

I don't think the MRN should change if it already has a value.

- If you're using 'toLowerCase()' then what does the 'i' regexp flag serve?

- Style: isn't $('#edit-name', context) redundant?

- This code converts "One (two)" to "one__two_". Those superfluous underscores aren't nice.

- This code converts "הכל בסבבה ואני בג'ננה" to "_________________". Ditto.

Rowanw:
It doesn't seem to happen to me. The 'type' only changes when I make a change to the 'name'. Hmm.. a class gets added that should stop it from changing if the js has changed it once. I'm not sure why this isn't working.

mooffie:
I also agree that if there was already something for the name field, it shouldn't be changed by js anymore. Any idea how I can add that into the JS? I have pretty limited js abilities. Can you suggest something to help internationalize it? Also can you suggest a way of avoiding the '__' issue?

Thanks

This patch should do the trick :) (based off qbnflaco's latest patch).

• only generates a new MRN if the field is blank
• suppresses duplicate underscores
• removes trailing underscores

Let me know if there's anything else I can do to help with this :)

No patch. :(

eek, I thought it went through last time. strange…

hopefully it works this time :)

It would be simpler to just do: replace(/[^a-z0-9]+/g, '_').replace(/^_+|_+$/, ''). This will also remove underscores at the beginning of the name.

Thanks for taking the time to report this issue.

However, marking as duplicate of #471018: Generic pattern/js for hiding machine readable names (applied to menu and content types).
You can follow up on that issue to track its status instead. If any information from this issue is missing in the other issue, please make sure you provide it over there.