Multi-Lingual Feedback Tab (MLFT) is a Free fully functional and easy to customize feedback tab by One Hour Translation.
MLFT allows any site owner to add multi-lingual feedback tab to any page in a simple and easy way.

Main Features:

  • Add multi-lingual feedback to any web-page easily
  • Free of charge
  • Multi Lingual - users can see the tab and provide the feedback in their language
  • Dynamic language selection - controlled by the admin
  • Users Feedback is sent by email and is also available online
  • Feedback is machine translated with Google and can be human translated by One Hour Translation.
  • Fields selection - admin has full control over the tab look and feel
  • Tab skins - Admin can fully customize the tab with any background you like
  • Generate as many tabs as you like
  • Complete Feedback management system

To use the Multi Lingual Feedback Tab:

  1. Download, install and activate the module
  2. Register to One Hour Translation (free) to open an account. The account will be used to manage the tab/s.
  3. Goto https://www.onehourtranslation.com/mlf/myForms
  4. Click "Create New Tab"
  5. Enter Tab details and click Save.
  6. Copy the Formid from the tab "Get Code" section.
  7. Paste the Formid to the configuration section on the Drupal module and click "save configuration

project page - http://drupal.org/sandbox/OneHourTranslation/1438382
Git - git clone http://git.drupal.org/sandbox/OneHourTranslation/1438382.git multi_lingual_feedback_tab
Version for Drupal 7

CommentFileSizeAuthor
#1 pareview_mlft.txt1.1 KBsvenaas

Comments

svenaas’s picture

StatusFileSize
new1.1 KB

As a fellow project applicant I thought I'd pull my weight and take a look. For starters I ran your project through ventral's online PAReview code checker. It found these problems:

  1. You're doing your work on the master branch rather than a version-specific branch. Here are some instructions on making the move.
  2. Your module lacks a README.txt file
  3. A few code-formatting and comment irregularities (see attachment).

I took a look at your code by hand as well. The module is pretty straightforward, using a couple of configuration settings to add a chunk of JavaScript to the bottom of the page. I do not see any obvious problems. I was pleased to see the t() calls where I expected them (especially given the nature of the project).

Robertas’s picture

Status: Needs review » Needs work

Marking as "needs work".

berdyshev’s picture

Status: Needs work » Needs review

We have fixed all this issues. Please review again.

targoo’s picture

Status: Needs review » Needs work

here my manual review

62 '#description' => t('If you don\'t have FormID, you should create new feedback form on www.onehourtranslation.com.'),
1) t() should only contain labels (no links)
2) do not use \'

57 '#default_value' => variable_get('mlft_enabled', 0),
3) // Form API will NOT sanitize the '#options' attribute on other elements than select boxes.
http://drupal.org/node/28984

4) You define getter but not setter. Why ?
variable_get('mlft_enabled', 0)

I am not sure the module offer yet all the features...

please check http://groups.drupal.org/node/195848

patrickd’s picture

@targoo
thank you for your efforts in the issue queue, but using more <code> tags for code would be great ;-)

DavidS’s picture

Status: Needs work » Closed (fixed)