[D7] Spam filter

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

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

1) Coding standard
Line 237: The $string argument to t() should not begin or end with a space. (Drupal Docs) [i18n_11]
drupal_set_message(t("Duplicate entry: unable to insert '@spam_text' ", array('@spam_text' => $spam_text)), 'error');

2) Line 103: Whats the reason for getting arguments trough $form_state['build_info']['args'], use this instead
function spam_filter_update_form($form, &$form_state, $spam_text_id) {

3) Not quite sure why you would use the object check in spam_filter_update_form that way and not just return nothing when there is no object.

there is some problem with ventral.org : http://ventral.org/pareview/httpgitdrupalorgsandboxdebtconsolidationcare...

I checked inside the module and I think they are true, the permission should be changed

37 | WARNING | The administration menu callback should probably use
| | "administer site configuration" - which implies the user can
| | change something - rather than "access administration pages"
| | which is about viewing but not changing configurations.
44 | WARNING | The administration menu callback should probably use
| | "administer site configuration" - which implies the user can
| | change something - rather than "access administration pages"
| | which is about viewing but not changing configurations.
53 | WARNING | The administration menu callback should probably use
| | "administer site configuration" - which implies the user can
| | change something - rather than "access administration pages"
| | which is about viewing but not changing configurations.
62 | WARNING | The administration menu callback should probably use
| | "administer site configuration" - which implies the user can
| | change something - rather than "access administration pages"
| | which is about viewing but not changing configurations.

1. The module duplicates the functionality that is already present in core (Trigger/Actions).
2. On the settings page, there is no cleaning after floating buttons (see attached screenshot).

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.