This module adds a new array in your images fields, containing absolute URLs to styled image of this field.
In administration, chose, for any content type, the formats for which you want the styled urls at node load.
Project page : https://drupal.org/sandbox/gregkapustin/2224501
View on Github : https://github.com/GregKapustin/Node-Load-Img
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | node_load_img-review.patch | 9.89 KB | sawtell |
Comments
Comment #1
PA robot commentedGit clone failed for http://git.drupal.org/sandbox/gregkapustin/2224501.git while invoking http://pareview.sh/pareview/httpgitdrupalorgsandboxgregkapustin2224501git
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.
Comment #2
sawtell commentedHi gregory_kapustin,
Your git repository is lacking a valid version branch, see this page for info on how to fix this. https://drupal.org/empty-git-master
The PAReview will likely highlight a number of formatting issues.
I have attached a patch that will fix the main ones although you will still need to rectify some. E.g. the PHPDoc comments in node_load_img.module line 46 - 50.
Also there are some lines that are over 80 chars long and will need to include a line break or re-worked.
See https://drupal.org/patch/apply to apply patches.
The indentation used did not match the Drupal coding standards, you may need to alter your text editor or IDE to mirror the standards that Drupal requires.
Drupal coding standards: https://drupal.org/coding-standards#indenting
Good luck with your application.
Edit: I've just tested the patch and it didn't apply the changes to the .info file.
The 'description' and 'core' definitions are missing in the info file. See https://drupal.org/node/542202
Comment #3
PA robot commentedClosing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.