[D7] Beevou

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

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.

Hi,

I had a quick review of you code and I have a few remarks about it.

First, it seems that even if you have a 7.x-1.x branch, your most up to date branch is still the "master" branch.
According to the branching convention, you should commit all your code on the 7.x-1.x branch.

I only had a look on the "master" branch.

First, you hardcoded both BEEVOU_CLIENT_ID and BEEVOU_CLIENT_SECRET ids. I don't know how it is supposed to work, but I guess that those need to be changed with personal credentials to each one that want to use your module ?

Also, you use cURL to query the service. That's fine, but what happen when a user want to use your module on a server that doesn't have cURL on it ? Maybe you should check that there is cURL enabled ? You can do that using hook_requirements for example. Check the color module (D7 core) for a live example, it checks if GD is present or not on the server.

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.