Project Name: Date Formatter
Drupal Version: 7.x
https://www.drupal.org/sandbox/alauddinthewonder/2498737

Git clone URL:
git clone --branch 7.x-1.x http://git.drupal.org/sandbox/AlauddinTheWonder/2498737.git date_formatter

Description:
Sometimes you need to create date input field but not restrict user to enter in a particular format. Yes it's now possible with the help of this module. This module let user's to input their date in any kind of form and when user blur the field it automatically convert date into admin's desired format. Let's take an example, suppose user input date like 31/12/2015 or 12/31/2015 or 2015/31/12 or 2015/12/31 or 31-12-2015 or 12-31-2015 or 2015-12-31 etc., this program will convert all of them into 12/31/2015 (as admin set it to mm/dd/yyyy format).

The Date Formatter module is a wrapper for the DateFormatter jQuery Plugin by Alauddin Ansari,
https://github.com/AlauddinTheWonder/jQuery-DateFormatter. It allows a user to more easily
enter any form of date and it'll converted into correct format.

CommentFileSizeAuthor
Date Formatter Settings My Drupal.png113.92 KBwonder_a7
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wonder_a7’s picture

wonder_a7 CreditAttribution: wonder_a7 as a volunteer and commented

Module has been installed on many projects and it's working fine.

Shamsher_Alam’s picture

Shamsher_Alam CreditAttribution: Shamsher_Alam commented
Status: Needs review » Needs work

FILE: /var/www/drupal-7-pareview/pareview_temp/dateformatter.module
---------------------------------------------------------------------------
FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES
---------------------------------------------------------------------------
50 | WARNING | Do not use the raw $form_state['input'], use
| | $form_state['values'] instead where possible
52 | WARNING | Do not use the raw $form_state['input'], use
| | $form_state['values'] instead where possible
54 | WARNING | Do not use the raw $form_state['input'], use
| | $form_state['values'] instead where possible
56 | WARNING | Do not use the raw $form_state['input'], use
| | $form_state['values'] instead where possible
72 | WARNING | #description values usually have to run through t() for
| | translation
80 | WARNING | #description values usually have to run through t() for
| | translation
122 | WARNING | Unused global variable $base_url.
---------------------------------------------------------------------------

Hi wonder_a7,

Thanks for contribution.i have reviewed your project there some standard issues in your module.You can also test its by self using this URL :
http://pareview.sh/pareview/httpgitdrupalorgsandboxalauddinthewonder2498737git

And one more thing please mention git detail in your issues for another user can take clone and review its easily.

Thanks,

Pravin Ajaaz’s picture

Pravin Ajaaz CreditAttribution: Pravin Ajaaz commented
Issue summary: View changes
Pravin Ajaaz’s picture

Pravin Ajaaz CreditAttribution: Pravin Ajaaz commented

Manual Review:

1. $base_url variable is never used in module file please remove it.
2. You should follow the drupal commenting standards for the hooks. eg: implements hook_block_info() .
3. Try to fix the indentation issues in pareview.
4. In js file you should use drupal behaviours.

PA robot’s picture

PA robot CreditAttribution: PA robot commented
Status: Needs work » Closed (won't fix)

Closing 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.