In 4.2 and CVS:

From node.module line 1424 till 1442:

      case "add":
        theme("box", t("Create new $name"), node_add(arg(2)));
        break;
      case "edit":
        theme("box", t("Edit $name"), node_edit(arg(2)));
        break;
      case "view":
        print node_show($node, arg(3));
        break;
      case t("Preview"):
        $edit = node_validate($edit, $error);
        theme("box", t("Preview $name"), node_preview($edit, $error));
        break;
      case t("Submit"):
        theme("box", t("Submit $name"), node_submit($edit));
        break;
      case t("Delete"):
        theme("box", t("Delete $name"), node_delete($edit));
        break;

These are making transalation much harder, because the $title will generate a new translatable string for every new node you submit...

Example:

      case t("Preview"):
        $edit = node_validate($edit, $error);
        theme("box", t("Preview $name"), node_preview($edit, $error));

Should be something like:

      case t("Preview"):
        $edit = node_validate($edit, $error);
        theme("box", t("Preview %title", array("%title" => "$name")), node_preview($edit, $error));

Comments

Stefan Nagtegaal’s picture

Well, the $name is the node-type which is being returned insteadof the node-title what I first thought. But still, I think it would be done another way..
My patch does the following:
You now can translate the "Preview %type", "Submit %type", "Delete %type", "Edit %type" and "Create new %type".. The '%type' wil be replaced by the translatable $node->type so it is much less to translate as before...

Patch sent to the list.
If it's applied, i'll close this bug-report....

moshe weitzman’s picture

Title: node.module » translations in node.module
Component: Node system » node system

no activity for a few weeks. please update the title of this item and provide a status update or else we shall close this one.

Anonymous’s picture

Fixed in CVS.

Anonymous’s picture

Automatically closed due to inactivity (marked fixed for 14 days).