Looking at the screenshot on the overview page for this project, I noticed one typo (or grammatical issue). The module looks interesting, so I thought I'd try it out. I've fixed a few odd strings and will add them as a patch here. In at least one case, rephrasing might be mostly to get documentation lines to 80 characters or less (coding standards), otherwise this patch is mostly just a few minor fixes for grammar/spelling/clarity.

CommentFileSizeAuthor
#1 fix_english_strings-1778454.patch2.92 KBlomo

Comments

lomo’s picture

lomo commented
StatusFileSize
new fix_english_strings-1778454.patch2.92 KB

And here is maybe the most trivial patch ever, but if you want to have grammatically correct strings it’s a good time to get them fixed (before translations get anchored to the strings which could use some tweaking).

Thank you for contributing this module. :-)

lord_of_freaks’s picture

lord_of_freaks commented

Thank you so much ... I comit the pAtch ASAP ... My english is horrible thanks for this great work

lord_of_freaks’s picture

lord_of_freaks commented
Assigned: lomo » lord_of_freaks
Status: Needs review » Patch (to be ported)

Patch apllied and working, it will be included in the next release

Thank you so much @LoMo.

lord_of_freaks’s picture

lord_of_freaks commented
Status: Patch (to be ported) » Closed (fixed)
lomo’s picture

lomo commented
Status: Closed (fixed) » Patch (to be ported)

No problem… you're most welcome and I'm glad you appreciate my small contribution. BTW, I'll be profiling “Required by role” among my selection for August’s “Modules of the Month” on the Cocomore Drupal Blog; the article should be live in the next day or two.

lord_of_freaks’s picture

lord_of_freaks commented
Status: Patch (to be ported) » Closed (fixed)