magicyril [magicyril]

1.       'description' => t('Bluetrip settings.'),
   

   Schema description should not be passed to t() anymore.

2.     $options['span-'. $i] = t('span '. $i);
   

   The first argument of t() is a literal string, not a dynamic value; differently, the script to extract the strings to translate will not extract the string.

3.   if ($values['css_classes']) {
       $value .= ' '. check_plain($values['css_classes']);
     }
     bluetrip_block_save($name, $value);
   

   check_plain() is used when showing a string, not when saving it.

4. The file LICENSE.txt needs to be removed; Drupal.org CVS doesn't allow to commit that file.

- You missed #1 above. it's in your hook_schema.

-You're still running check_plain() on input; you just moved it to a different function. check_plain() should only be run on output.

Please read the following links as this is very important information about CVS applications.

Drupal.org has moved from CVS to Git! This is a very significant change for the Drupal community and for these applications. Please read Migrating from CVS Applications to (Git) Full Project Applications and Applying for permission to opt into security advisory coverage on how this affects and benefits you and the application process. In short, every user has now the permissions necessary to create new projects, but they need to apply for opt into security advisory coverage. Without applying, the projects will have a warning on projects that says:

This project is not covered by Drupal’s security advisory policy.