Biblio Mendeley Sync for D7 [#1426462]

This module is an add-on for the popular Biblio module that allows you to sync select biblio records with the online research tool, Mendeley (http://mendeley.com). There is a Drupal 6 version on Github (https://github.com/jakob-stoeck/drupal-biblio-mendeley), but I went ahead and refactored it into a Drupal 7 version, and updated the functionality along the way.

Florida State University is sponsoring development of this project, and uses it for the MSRC website (http://msrc.fsu.edu).

Link to project page: http://drupal.org/sandbox/caseyamcl/1393134
Link to GIT repository: git.drupal.org:sandbox/caseyamcl/1393134.git
Drupal Version: 7.x

Comments

Comment #1

eugene.ilyin commented 5 February 2012 at 19:24

Hello!
I spent a Review of your module.

1. Please correct your address repository git.drupal.org: sandbox/caseyamcl/1393134.git, and add a reference indicating the branches for git commands such as "git clone --branch 7.x-1.x http://git.drupal.org/sandbox/caseyamcl/1393134.git biblio"
2. There are still files other than README.txt in the master branch, make sure to remove them. See also step 5 in http://drupal.org/node/1127732
Review of the 7.x-1.x branch:

README.txt is missing, see the guidelines for in-project documentation.
The "?>" PHP delimiter at the end of files is discouraged, see http://drupal.org/node/318#phptags
```
./includes/oauth.inc
```
./biblio_mendeley.install: all functions should be prefixed with your module/theme name to avoid name clashes. See http://drupal.org/node/318#naming
```
function _biblio_installed_instances() {
```
./biblio_mendeley.module: all functions should be prefixed with your module/theme name to avoid name clashes. See http://drupal.org/node/318#naming
```
function Biblio_mendeley_obj($obj) {
```
Bad line endings were found, always use unix style terminators. See http://drupal.org/coding-standards#indenting
```
./.gitignore:                                        ASCII text, with no line terminators
```
Run coder to check your style, some issues were found (please check the Drupal coding standards). See attachment.
Drupal Code Sniffer has found some code style issues (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. Get a review bonus and we will come back to your application sooner.