Project page: http://drupal.org/sandbox/mishac/1539958
GIT: git.drupal.org:sandbox/mishac/1539958.git
Drupal version: 7.x

This module allows you to create bookmarklets that let you quickly create Drupal nodes from external web pages, with the title, body, link, and image fields prepopulated. It allows you to choose which images to insert into your post, similar to when you share a link on Facebook or Google+.

It also optionally uses the PHP Readability library (http://www.keyvan.net/2010/08/php-readability/) to more intelligently get the content and title of an article, instead of just relying on the content of the HTML title and body tags.

There are other bookmarklet modules, but they have limitations:

1) Bookmarklet (http://drupal.org/project/bookmarklet) hasn't been updated for over a year, doesn't work with Drupal 7, and doesn't support images.

2) Quickpost Bookmarklet (http://drupal.org/project/quickpost_bookmarklet) doesn't support grabbing images or using the readability library to intelligently grab the content text of a page.

Comments

patrickd’s picture

Status: Needs review » Needs work

welcome,

Why not taking over / joining forces with one of this existing projects ?
It would be better to have it all in one place instead of creating yet-another bookmarklet module.

You are working in a wrong-named "7.x-0.x-dev" branch in git, with this name you wont be able to create releases later. 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.

As installation and usage instructions are quite important for us to review, please take a moment to make your project page follow tips for a great project page. Also create a README.txt that follows the guidelines for in-project documentation.

while waiting for an in-depht review of your module you can start out fixing some coding style issues detected by automated tools:
http://ventral.org/pareview/httpgitdrupalorgsandboxmishac1539958git-7x-0... you can retest yourself at http://ventral.org/pareview after you switched the branch.

We do really need more hands in the application queue and highly recommend to get a review bonus so we can come back to your application sooner.

regards

mishac’s picture

I've fixed the branch name as well as cleaned up the code to clear up all the style issues in the automated ventral report.

The reason I created a new module is that

a) the bookmarklet module appears essentially abandoned

b) The quickpost module, and the prepopulate module it relies on, use $_GET queries to fill in data in the node form, which doesn't allow for pulling data in non-text fields, like images, media, files etc.

What I'm going for is more like the facebook share-link experience than just filling in some text fields, to make it simple for users to create nodes like blog posts that reference external content.

patrickd’s picture

Status: Needs work » Postponed

Well,
a) Taking over an abandoned module
b) Applying for co-maintainership
Please prefer one of these roads first before we continue here

Daidy’s picture

There is no good bookmarklet for Drupal at this time. I really don't see the need to discourage this guy...
Please go on mishac, and do a new and good module if you want to.

Diogenes’s picture

Status: Postponed » Needs review

@Daidy - Hear, hear!
@patrickd
Patrick, you once gave me the same advice. In my particular case, the Members module was abandoned over 4 years ago, with the last code for versions 4.7 and 5. And the naming conflict that drush users get their knickers in a knot about makes not a whit of difference as to whether the module actually works or not.

I did not say it then but I'll say it now. That advice was not needed and was highly offensive. Drop it from your repertoire of stock responses. It is incredibly discouraging.

The current Drupal community really needs to clean up the cruft on its website instead of lecturing potential new contributors about the proper protocols (no matter how irrelevant) for joining the club.

Setting back to 'needs review' based on Daidy's comment, just wish I could bump it back up to the top.

greggles’s picture

Status: Needs review » Postponed

I think one thing that a lot of people mistake is that taking over an existing namespace is extremely beneficial for a new module maintainer. Expisting projects come with:

* Search engine visiblity
* User bases of people interested in a similar solution

If you can get a fast forward on your project's progress, why not take it? Yes, it takes a little effort (and two weeks) to take over an abandoned module. But getting search engine visibility and userbase usually takes much longer than 2 weeks.

fizk’s picture

Hi mishac, are you still interested in promoting your sandbox project to a full project?

klausi’s picture

Status: Postponed » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

klausi’s picture

Issue summary: View changes

fix url for quickpost bookmarklet module