The linked file is a modified version of the defaultcontent.module file that fixes:

  • Error when updating a machine name. The module did do nothing
  • Error when deleting a machine name. The module did nothing.
  • Adding a function that checks if a machine name already exists.

It also adds the permission patch. I also reformatted most of the function comments and marked some functions deprecated (defaultcontent_menu).

Maybe we can use this as a start to incorporate other patches and then use this for another release? I can create patches for the fixes in the mean time as well.

What's left to do?

  • Check if the deprecated code can be deleted
  • Finish to add comments on each function
  • Have a look at the @todo in the module
  • Add ctools as an dependency in the info file

Link to Gist: https://gist.github.com/1425366

Comments

Jeffrey C.’s picture

Jeffrey C. CreditAttribution: Jeffrey C. commented

Thank you so much. But could you submit a patch so I can commit it? I can go through your code but it'd take some more time to do so.