The node delete module in Drupal 7 adds the delete menu item as an action.
Just enable and the delete link will appear in the tab of actions.

Link to page : http://drupal.org/sandbox/CloneVince/1923494
Link to repository : git clone --recursive --branch master CloneVince@git.drupal.org:sandbox/CloneVince/1923494.git node_delete_tab

This module is for Drupal 7

It's my first module, i hope i did all what was needed

Comments

Propaganistas’s picture

Hi

You're code is far below the required 120 lines of code to be qualified as a full project... This should rather go in some sort of custom functions module.

Anyhow, there are still a few issues regarding your project:

  • You're using a master branch. Move your code repository to a custom branch like 7.x-1.x, then delete master branch.
  • Some indentation and whitespace issues. Please refer to the Coding standards.
  • The function you use should be documented more clearly. Please refer to the Coding standards.
  • Delete the LICENSE.txt file. This file will be automatically added by Drupal when you promote the project to a full release.
  • There's no need to declare your .module file in the .info.
klausi’s picture

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 :-)

davidmac’s picture

Status:Needs review» Needs work

Hi,

Please review the results of the pareview script and refer to the coding standards as mentioned in comment #1 above:

FILE: /tmp/pareview_temp/node_delete_tab.info
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
5 | ERROR | It's only necessary to declare files[] if they declare a class or
   |       | interface.
--------------------------------------------------------------------------------

FILE: /tmp/pareview_temp/node_delete_tab.module
--------------------------------------------------------------------------------
FOUND 11 ERROR(S) AND 1 WARNING(S) AFFECTING 5 LINE(S)
--------------------------------------------------------------------------------
1 | ERROR   | Missing file doc comment
4 | WARNING | Format should be "* Implements hook_foo().", "* Implements
   |         | hook_foo_BAR_ID_bar() for xyz_bar().", or "* Implements
   |         | hook_foo_BAR_ID_bar() for xyz_bar.tpl.php.".
4 | ERROR   | Function comment short description must start with a capital
   |         | letter
4 | ERROR   | Function comment short description must end with a full stop
5 | ERROR   | There must be an empty line before the parameter block
5 | ERROR   | Expected a valid @param data type, but found unknown_type
5 | ERROR   | Missing comment for param "$items" at position 1
8 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
8 | ERROR   | Line indented incorrectly; expected 2 spaces, found 1
8 | ERROR   | Inline comments must end in full-stops, exclamation marks, or
   |         | question marks
9 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
9 | ERROR   | Line indented incorrectly; expected 2 spaces, found 1
--------------------------------------------------------------------------------

mikespence’s picture

Manual Review

  • You should remove the master branch and create a working branch.
  • Update your git command to: git clone --recursive --branch master http://git.drupal.org/sandbox/CloneVince/1923494.git node_delete_tab
  • Remove your LICENCE.txt file - this is generated by Drupal
  • Check for code errors here: http://ventral.org/pareview/httpgitdrupalorgsandboxclonevince1923494git
klausi’s picture

@mikespence: this application is already in the "needs work" state. Please review applications that are in the "needs review" state. See also http://drupal.org/node/532400

ycshen’s picture

you should add the readme.txt file

PA robot’s picture

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.