diff --git a/includes/nodeownership.admin.inc b/includes/nodeownership.admin.inc
index 0e67fbd..c4327c7 100644
--- a/includes/nodeownership.admin.inc
+++ b/includes/nodeownership.admin.inc
@@ -1,9 +1,14 @@
 <?php
 
 /**
+ * @file
+ * Contains code related to configurations for admin interface.
+ */
+
+/**
  * Implementation of admin setting page.
  *
- * @return
+ * @return array $form
  *   array of form elements
  */
 function nodeownership_admin_settings($form, &$form_state) {
@@ -28,7 +33,6 @@ function nodeownership_admin_settings($form, &$form_state) {
     '#default_value' => variable_get('nodeownership_pending_text', 'Claim Pending'),
   );
 
-
   $form['email'] = array(
     '#type' => 'fieldset',
     '#title' => t('E-mail settings For Node Ownership'),
diff --git a/includes/nodeownership.inc b/includes/nodeownership.inc
index 8172d2a..5834efd 100644
--- a/includes/nodeownership.inc
+++ b/includes/nodeownership.inc
@@ -29,7 +29,15 @@ function nodeownership_claim_approve($claim) {
     ->execute();
 
   nodeownership_send_mail('approve_accept', $claim->uid);
-  watchdog('nodeownership', 'Approved claim @claim_id by user @uid for node @nid', array('@claim_id' => $claim->id, '@uid' => $claim->uid, '@nid' => $claim->nid));
+  watchdog(
+    'nodeownership',
+    'Approved claim @claim_id by user @uid for node @nid',
+    array(
+      '@claim_id' => $claim->id,
+      '@uid' => $claim->uid,
+      '@nid' => $claim->nid,
+    )
+  );
 
   return $num_updated ? TRUE : FALSE;
 }
@@ -53,14 +61,23 @@ function nodeownership_claim_decline($claim) {
     ->execute();
 
   nodeownership_send_mail('approve_decline', $claim->uid);
-  watchdog('nodeownership', 'Declined claim @claim_id by user @uid for node @nid', array('@claim_id' => $claim->id, '@uid' => $claim->uid, '@nid' => $claim->nid));
+  watchdog(
+    'nodeownership',
+    'Declined claim @claim_id by user @uid for node @nid',
+    array(
+      '@claim_id' => $claim->id,
+      '@uid' => $claim->uid,
+      '@nid' => $claim->nid,
+    )
+  );
 
   return $num_updated ? TRUE : FALSE;
 }
 
 /**
- * Sets parameters for outgoing mail and execute mail delivery
- * for admin
+ * Sets parameters for outgoing mail.
+ *
+ * Execute mail delivery for admin.
  */
 function nodeownership_send_mail($type, $uid) {
   global $language;
@@ -81,14 +98,16 @@ function nodeownership_send_mail($type, $uid) {
 }
 
 /**
- * Function for setting default mail content and saving custom mail content by admin.
+ * Function for setting default mail content.
+ *
+ * Saving custom mail content by admin.
  *
- * @param $key
- *   parameter for different mail content.
- * @param $language
+ * @param string $key
+ *   Parameter for different mail content.
+ * @param bool $language
  *   It is optional.
- * @param $variables
- *   a set of variables to be used inside mail content.
+ * @param array $variables
+ *   A set of variables to be used inside mail content.
  *
  * @return string
  *   it returns the subject and body of different mail.
diff --git a/includes/nodeownership.pages.inc b/includes/nodeownership.pages.inc
index b6eadc6..447ab78 100644
--- a/includes/nodeownership.pages.inc
+++ b/includes/nodeownership.pages.inc
@@ -1,12 +1,18 @@
 <?php
 
 /**
+ * @file
+ * Contains callback functions.
+ */
+
+/**
  * Implementation of logic for user claim.
  *
  * @param object $node
  *   Claimed node.
  *
  * @return array
+ *   Returns node ownership claim form for particular node.
  */
 function nodeownership_get_claim($node) {
   global $user;
@@ -35,13 +41,13 @@ function nodeownership_get_claim($node) {
 /**
  * Implementation of the use claim form.
  *
- * @param $form_state
- *   The form state
- * @param $nid
+ * @param array $form_state
+ *   The form state.
+ * @param int $nid
  *   Node Id of the claimed node.
  *
- * @return array
- *   return form element.
+ * @return array $form
+ *   Return form element.
  */
 function nodeownership_claim_form($form, &$form_state, $nid) {
   $form          = array();
@@ -77,10 +83,10 @@ function nodeownership_claim_form($form, &$form_state, $nid) {
 /**
  * Implementation for user claim submission.
  *
- * @param $form
- *   form filled by user
- * @param $form_state
- *   state of the form submitted.
+ * @param array $form
+ *   Form filled by user.
+ * @param array $form_state
+ *   State of the form submitted.
  */
 function nodeownership_claim_form_submit($form, &$form_state) {
   global $user;
@@ -98,15 +104,13 @@ function nodeownership_claim_form_submit($form, &$form_state) {
   return drupal_goto('node/' . $form_state['values']['nid']);
 }
 
-/*
+/**
  * Implementation of logic for claim confirmation by admin(approve/reject).
  *
- * @param $type
- *   confirmation type approve/decline.
- * @param $nid
- *   nid of the node.
- * @param $uid
- *   uid of the claimed user.
+ * @param string $type
+ *   Confirmation type approve/decline.
+ * @param int $id
+ *   Nid of the node.
  */
 function _node_claim_process($type, $id) {
   module_load_include('inc', 'nodeownership', 'includes/nodeownership');
diff --git a/includes/nodeownership.views.inc b/includes/nodeownership.views.inc
index 2646730..018bbb7 100644
--- a/includes/nodeownership.views.inc
+++ b/includes/nodeownership.views.inc
@@ -6,13 +6,12 @@
  */
 
 /**
- * Implements hook_views_data()
+ * Implements hook_views_data().
  */
 function nodeownership_views_data() {
 
   // ----------------------------------------------------------------
   // nodeownership table -- basic table information.
-
   $data['nodeownership']['table']['group'] = t('Node ownership');
   $data['nodeownership']['table']['base'] = array(
     'field' => 'id',
@@ -31,8 +30,7 @@ function nodeownership_views_data() {
   );
 
   // ----------------------------------------------------------------
-  // nodeownership -- fields
-
+  // nodeownership -- fields.
   // Uid.
   $data['nodeownership']['uid'] = array(
     'title' => t('User uid'),
@@ -166,4 +164,3 @@ function nodeownership_views_data() {
 
   return $data;
 }
-
diff --git a/nodeownership.info b/nodeownership.info
index 5f78baa..55c342d 100644
--- a/nodeownership.info
+++ b/nodeownership.info
@@ -13,4 +13,4 @@ files[] = includes/views_handler_field_nodeownership_status.inc
 files[] = includes/views_handler_filter_nodeownership_status.inc
 files[] = includes/views_handler_field_nodeownership_approve_link.inc
 files[] = includes/views_handler_field_nodeownership_decline_link.inc
-configure = admin/config/nodeownership/settings
\ No newline at end of file
+configure = admin/config/nodeownership/settings
diff --git a/nodeownership.install b/nodeownership.install
index dbd3d94..1bcfc3f 100644
--- a/nodeownership.install
+++ b/nodeownership.install
@@ -11,7 +11,7 @@
 function nodeownership_install() {
   $node_types = node_type_get_names();
   $all_node_types = array();
-  foreach ($node_types AS $key => $value) {
+  foreach ($node_types as $key => $value) {
     $all_node_types[$key] = 0;
   }
   variable_set('node_types', $all_node_types);
@@ -101,7 +101,7 @@ function nodeownership_update_7100(&$sandbox) {
   if (empty($get_node_types)) {
     $node_types = node_type_get_names();
     $all_node_types = array();
-    foreach ($node_types AS $key => $value) {
+    foreach ($node_types as $key => $value) {
       $all_node_types[$key] = 0;
     }
     variable_set('node_types', $all_node_types);
@@ -123,8 +123,7 @@ function nodeownership_update_7120(&$sandbox) {
     'primary key' => array('id'),
   );
   db_add_field('nodeownership', 'id', $spec, $keys_new);
-  
+
   // Clear caches to load new menu items.
   cache_clear_all();
 }
-
diff --git a/nodeownership.module b/nodeownership.module
index 69d139d..0736ec6 100644
--- a/nodeownership.module
+++ b/nodeownership.module
@@ -6,19 +6,19 @@
  */
 
 /**
-  * Implements hook_views_api().
-  */
+ * Implements hook_views_api().
+ */
 function nodeownership_views_api() {
- return array(
-   'api' => 3,
-   'path' => drupal_get_path('module', 'nodeownership') . '/includes',
- );
+  return array(
+    'api' => 3,
+    'path' => drupal_get_path('module', 'nodeownership') . '/includes',
+  );
 }
 
 /**
  * Implements hook_help().
  */
-function nodeownership_help($page = 'admin/config/help#nodeownership', $arg) {
+function nodeownership_help($arg, $page = 'admin/config/help#nodeownership') {
   switch ($page) {
     case 'admin/config/help#nodeownership':
       return t('<p>The nodeownership module allows you to configure a node such that it can be claimed by some user. Once the user has granted the ownership
@@ -46,7 +46,7 @@ function nodeownership_permission() {
     'show claim link' => array(
       'title' => t('Show claim link'),
       'description' => t('Allows a user to see a claim link.'),
-    )
+    ),
   );
 }
 
@@ -116,9 +116,9 @@ function nodeownership_node_view($node, $view_mode = 'full') {
     // Check to see if the current user has a pending claim.
     $query = new EntityFieldQuery();
     $query->entityCondition('entity_type', 'claim')
-    ->propertyCondition('uid', $user->uid)
-    ->propertyCondition('nid', $node->nid)
-    ->propertyCondition('status', 0);
+      ->propertyCondition('uid', $user->uid)
+      ->propertyCondition('nid', $node->nid)
+      ->propertyCondition('status', 0);
     $num_claims = $query->count()->execute();
 
     // Build dummy link for pending claims.
@@ -134,7 +134,7 @@ function nodeownership_node_view($node, $view_mode = 'full') {
   // Display link.
   $node->content['links']['nodeownership'] = array(
     // @todo make link output themable.
-    //'#theme' => 'links__node__flag',
+    // '#theme' => 'links__node__flag',
     '#links' => array(
       'claim-node' => array(
         'title' => $link,
@@ -153,7 +153,7 @@ function nodeownership_claim_insert() {
   global $user;
   global $language;
 
-  // Code For sending mail to Admin
+  // Code For sending mail to Admin.
   $recipient['mail'] = db_query("SELECT mail FROM {users} WHERE uid = 1")->fetchField();
   $recipient['uid']  = 1;
   $recipient['name'] = db_query("SELECT name FROM {users} WHERE uid = 1")->fetchField();
@@ -162,7 +162,7 @@ function nodeownership_claim_insert() {
 }
 
 /**
- * Implements of hook_mail().
+ * Implements hook_mail().
  */
 function nodeownership_mail($key, &$message, $params) {
   module_load_include('inc', 'nodeownership', 'includes/nodeownership');
@@ -175,8 +175,8 @@ function nodeownership_mail($key, &$message, $params) {
 }
 
 /**
-* Implements hook_entity_info().
-*/
+ * Implements hook_entity_info().
+ */
 function nodeownership_entity_info() {
   return array(
     'claim' => array(
@@ -189,7 +189,7 @@ function nodeownership_entity_info() {
       ),
       // Make use the class' label() and uri() implementation by default.
       'label callback' => 'entity_class_label',
-      //'uri callback' => 'entity_class_uri',
+      // 'uri callback' => 'entity_class_uri',.
     ),
   );
 }
@@ -217,9 +217,9 @@ function nodeownership_action_info() {
 /**
  * Action function for nodeownership_decline_claim_action.
  *
- * @param $entity
+ * @param object $entity
  *   Entity object.
- * @param $context
+ * @param array $context
  *   Array with parameters for this action: depends on the trigger.
  */
 function nodeownership_decline_claim_action(&$entity, $context = array()) {
@@ -231,9 +231,9 @@ function nodeownership_decline_claim_action(&$entity, $context = array()) {
 /**
  * Action nodeownership_approve_claim_action.
  *
- * @param $entity
+ * @param object $entity
  *   Entity object.
- * @param $context
+ * @param array $context
  *   Array with parameters for this action: depends on the trigger.
  */
 function nodeownership_approve_claim_action(&$entity, $context = array()) {