"Add new item" feature produces bad URLs for CCK content types

Also having the same problem.

I did a quick look in the code and came up with this hack, which seemed to work. (Sorry, don't know how to make patch files yet. :P)

You're looking for the function "workspace_add_form_submit" and the line that looks like:

    if (isset($options[$node_type])) {
      drupal_goto('node/add/' . $node_type);
    }

You'd modify it to look like:

    if (isset($options[$node_type])) {
	  /* Start hack */
	  if (stristr($node_type,"_") > -1) {
		$node_type = str_replace("_","-",$node_type);
	  }
	  /* End hack */
      drupal_goto('node/add/' . $node_type);
    }

Seems pretty simple...I think?

Thanks for the hack! Has anyone tested the code yet?

Frank

I've been using this little hack on a production environment for the past few months, haven't heard of any issues or complaints from my users.

Thanks for the feedback. Will include the patch in the next commit to HEAD.

Frank

btw, it's not really a hack. it's exactly what node.module (in core) does:

  foreach (node_get_types('types', NULL, TRUE) as $type) {
    $type_url_str = str_replace('_', '-', $type->type);
    $items['node/add/'. $type_url_str] = array(
...

The check for underscores should not be included. If there are no underscores, the check will take exactly as long as no check. If there are underscores, it will take about twice as long as it scans the string twice.

I hope to see this in a release soon as it's a very simple fix to a problem that is very confusing to users.

Sorry for the duplicate Issue report. I should have checked before. My fault. http://drupal.org/node/616462

This is my patch for a proper Workspace behavior :

     if (isset($options[$node_type])) {
-      drupal_goto('node/add/' . $node_type);
+      drupal_goto('node/add/' . str_replace('_', '-', $node_type));
     }

The actual node add url with Workspace :
example.com/node/add/node_type [WRONG]

With this patch :
example.com/node/add/node-type [GOOD]

Hi Michel,

Thanks for your patch. Have you tried the Dev version? The above mentioned hack should be already incorporated there.

Cheers,
Frank

Good to hear this issue have been fixed.

Could you release a "non-dev" version with this issue fixed as well? Just so it would look a bit nicer for anyone else stumbling upon this module. :)

-Are