Baidu Share module provides a Baidu Share buttons to nodes and as a separate block. Baidu Share is a popular button embed code for China and provides embed buttons for QQ, Weibo, RenRen and other China social networks.

Project Page: https://drupal.org/sandbox/jackniu/2090675

Version: Drupal 7.x-1.0
Git: git clone --branch master jackniu@git.drupal.org:sandbox/jackniu/2090675.git baidu_share

Files: 
CommentFileSizeAuthor
baidu_share.png9.99 KBjackniu

Comments

Status:Needs review» Needs work

You need add a 7.x-1.x branch and delect the master branch

There is still a master branch, make sure to set the correct default branch: http://drupal.org/node/1659588 . Then remove the master branch, see also step 6 and 7 in http://drupal.org/node/1127732
Review of the 7.x-1.x branch:

  • Bad line endings were found, always use unix style terminators. See http://drupal.org/coding-standards#indenting
    ./README.txt:       ASCII text, with CRLF line terminators
    README.txt
  • Coder Sniffer has found some issues with your code (please check the Drupal coding standards).
    FILE: /var/www/drupal-7-pareview/pareview_temp/README.txt
    --------------------------------------------------------------------------------
    FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
    --------------------------------------------------------------------------------
    7 | ERROR | Files must end in a single new line character
    --------------------------------------------------------------------------------

    FILE: /var/www/drupal-7-pareview/pareview_temp/baidu_share.info
    --------------------------------------------------------------------------------
    FOUND 2 ERROR(S) AFFECTING 1 LINE(S)
    --------------------------------------------------------------------------------
    1 | ERROR | "description" property is missing in the info file
    1 | ERROR | "core" property is missing in the info file
    --------------------------------------------------------------------------------

Thanks for dansboy's and qinshuguo's review, I have delete the master branch and add description and core to the info file.

Issue summary:View changes
Status:Needs work» Closed (won't fix)

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.