psy [psy]

The code to review must be uploaded here.

Until the status is not changed, you don't need to do anything.

I meant until the status is not changed from somebody who reviewed your module.
If nobody has made a review of your code that doesn't mean you need to post a better version.

The process is made only for the CVS account application; once you get it, the process is not repeated for each project you make.

1.   '#default_value' => check_plain(variable_get('uc_review_extras_refunds_title', 'Refunds')),
   

   The form API already filters the values used for the form fields.

2. $form['refunds']['uc_review_extras_refunds_text'] = array(
     '#input_format' => '1',
     '#rows' => '5',
     '#cols' => '40',
     '#weight' => '1',
     '#type' => 'textarea',
     '#title' => t('Text'),
     '#default_value' => variable_get('uc_review_extras_refunds_text', 'Refunds note here'),
   );
   

   #input_format is not an attribute normally used from Drupal; if you are trying to set the input format accepted, you should do something different.

3.   '#summary callback' => 'summarize_checkbox',
     '#summary arguments' => array(
         t("The '@title' notice is enabled.", array('@title' => variable_get('uc_review_extras_delivery_title', ''))),
         t("The '@title' notice is disabled.", array('@title' => variable_get('uc_review_extras_delivery_title', ''))),
       ),
   

   Also this attributes are not used by Drupal.

4. function uc_review_extras_install() {
   
     // init all the variables
     variable_set('uc_review_extras_delivery_title', 'Delivery');
     variable_set('uc_review_extras_security_title', 'Security');
     variable_set('uc_review_extras_privacy_title', 'Privacy');
     variable_set('uc_review_extras_returns_title', 'Returns');
   

   Drupal variables are not initialized during installation of a module; if you need a default value, that is the second argument passed to variable_get().

Remember to change the status of the report, after you upload new code; differently the reviewers (who are more than one) cannot know there is something to review.

1. The file LICENSE.txt needs to be removed.

2.     'description' => 'Edit the Review Extra Policy Statements.',
   

   Description and title doesn't use the title case; just the first word is written in capital case.

3.   if (variable_get('uc_review_extras_enabled', TRUE) == 1) return;
   

   Even in the simpler IF-statement, the coding standards suggests to use the curly brackets; the line should be written as

     if (variable_get('uc_review_extras_enabled', TRUE) == 1) {
       return;
     }
   

When new version planned to release?