A drupal 7 module and some associated elisp code that will allow you to manage drupal content in emacs org-mode, be aware that if you use this module to manage a content type, you cannot make any changes to that content through drupal.

FEATURES

* Manage a drupal content type with a directory of org files
* CCK Support
* Taxonomy Support
* Push based publishing (your site updates in real time)
* Node image Support
* URL Alias support
* Uses org generated html for body of post

LINKS
* http://www.gnu.org/software/emacs/
* http://orgmode.org/

PROJECT PAGE
* http://drupal.org/sandbox/math0ne/1826414

GIT
* git clone --recursive --branch 7.x-1.x math0ne@git.drupal.org:sandbox/math0ne/1826414.git drupal_org

Comments

math0ne’s picture

math0ne commented

I found this code review page: http://ventral.org/pareview

I fixed all the issues that the tool found: http://ventral.org/pareview/httpgitdrupalorgsandboxmath0ne1826414git

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented

We are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)

paravibe’s picture

paravibe commented

Please specify a right git link, something like this git clone --branch 7.x-1.x http://git.drupal.org/sandbox/math0ne/1826414.git drupal_org. Now it request your password.

Manual review:
drupal_org.info: line2, remove brackets.
drupal_org.module: line 21, specify a title for this page.
line 245, remove unnessesary ending.
line 236, your code $node->{$org_cck_field[0]}["und"][0]["value"] = $org_cck_field[1];. You used 'und' here and it mean that a language for this field is undefined and it will not be working for multilanguage site. Maybe you should take a field language and use it. E.g. $node->{$org_cck_field[0]}[$field_language][0]["value"] = $org_cck_field[1];

pere orga’s picture

pere orga
Primary language Catalan
commented

Why not call it "Org-mode " instead of "Drupal-Org"?

udaksh’s picture

udaksh commented

Hi math0ne,

Manual Review :

At Line 244 and 248. Use t() in place of check_plain().
You are currently using - drupal_set_message(check_plain("Unable to to save node: " . $org_title));
It would be better if you use t() with @ placeholder like
drupal_set_message(t('Unable to to save node: @org_title',array('@org_title' => $org_title)));

Thanks

sadashiv’s picture

sadashiv commented
Status: Needs review » Needs work

Implements hook_cron on line 264 shows drupal_org_example_cron() should be drupal_org_cron().

It will be nice if you implement hook_help to provide help for the module.

As specified in #5 should use t() instead of check_plain();

Regards,
Sadashiv.

PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

I'm a robot and this is an automated message from Project Applications Scraper.