A module that is using the new Twitter api (v1.1, using OAUTH) to fetch and store a selected nr of tweets on every cron-run to the drupal database and displays them in a block.

So if the api changes or can't be reached we still can display the latest tweets it fetched.

Link to Projectpage: https://drupal.org/sandbox/pixelpiloten/2036687
Link to Git-repository: http://git.drupal.org/sandbox/pixelpiloten/2036687.git

CommentFileSizeAuthor
Screenshot of the configuration page.103.62 KBpixelpiloten

Comments

PA robot’s picture

Status: Needs review » Needs work

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

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.

ayesh’s picture

Welcome!
Could you explain how your module differs from the following similar ones ?
- https://drupal.org/node/2046035
- https://drupal.org/sandbox/craigweb/1818578 / https://drupal.org/node/1818974
- https://drupal.org/project/twitter_block

Thanks.

garnett2125’s picture

Hi,

Manual review :

  1. In hook_menu() why don't you use drupal_get_form as page callback and the name of the form as page arguments, see https://api.drupal.org/comment/24118#comment-24118
  2. drupal_set_message('Twittercache did not detect the library TwitterAPIExchange.','error'); => all user facing text must run through t() for translation
  3. '#markup' => 'foo', Why do you put 'foo' as markup of your block ? Seems like a copy/paste of an example. You should use markup to put the markup of your block
  4. $html = '';, this variable is not used anywhere twittercache_get_cached_tweets()
PA robot’s picture

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.