[D6] Send Quote-- from every page

There are some errors reported by automated review tools, did you already check them? See http://ventral.org/pareview/httpgitdrupalorgsandboxsudiptapp2029583git

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then 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.

Closing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).

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

sudiptapp: please review the Drupal.org project application workflow for information on status that can be applied to a project application.

("Active" isn't part of the workflow at all, if the goal here was to get this up for review again "Needs Review" is the option you're looking for.)

From pareview.sh:

It appears you are working in the "6.x" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.
Review of the 6.x branch:

Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.

manual review:

• *.bak files should be removed. They serve no purpose.
• Function end doc blocks aren't needed (ie send_quote.admin.inc:115-117)
• send_quote.module:send_quote_block() - Might be better to move contact div to theme function/template so that styling, etc aren't hard-coded.
• send_quote.module:send_quote_formblock_block() - Inline styling will really limit other's ability to override if needed. consider moving to css stylesheet

The automated review of your project by PAReview has found some issues with your code. As coding standards make sure projects are coded in a consistent style we ask you to please have a look at the report and try to fix them.

Master Branch

It appears you are working in the "6.x" branch in git. That does not follow the branch naming conventions. Please see the documentation about release naming conventions and creating a branch in git.

Git Clone Command

There is no git clone command in your issue summary (only a link to the git repo).

Project page

Please take a moment to make your project page follow tips for a great project page. In particular you need to make it clear why this module is useful, and who it is useful for. As it stands, I don't understand why I would want to use this module on my site.

README.txt

Please take a moment to make your README.txt follow the guidelines for in-project documentation and the README.txt Template.

Coding standard

As noted above, the automatic review (PAReview has found some issues with your code). However, the manual code review reveals major differences from Drupal coding standards. It is hard to review non-standard code, so not conforming to Drupal coding standards lessens your chances of getting a manual code review (there are a lot of projects in the review queue, and at least I prefer those with code that is standard, clear, and simple to read).

Also, this project is for Drupal 6. Drupal 6 is about to become obsolete, and it may soon be hard to find somebody that still has a Drupal 6 test site set up for review purposes. You may consider upgrading upgrading the project to Drupal 7 or Drupal 8 to reach out to a larger audience of potential reviewers.

I am setting status to "Needs work". Please do not change the status to "Needs review" until you think you've addressed all the issues in the automatic review, the review by ericthelast, and this review.

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

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