Time to Date Plugin

Please Apply the Here attached Patch and this patch for port the Time to Date Plugin from D7 to D8.

Thanks @sivaramakrishnan - this one will also need a test before we can commit.

Hi @ericgsmith,

Here is the updated patch includes the test for time to date plugin, schema updates and fixes on the plugin code.

Thanks
Vijay

Coding standard fix applied to test file

Thanks Vijay - few minor comments.

1. +++ b/src/Plugin/Tamper/TimeToDate.php
   @@ -0,0 +1,72 @@
   +      $form_state->setErrorByName(self::SETTING_DATE_FORMAT, $this->t('Please enter a valid date format string.'));
   

   Setting the form to required would make this redundant.

2. +++ b/src/Plugin/Tamper/TimeToDate.php
   @@ -0,0 +1,72 @@
   +    return date($this->getSetting(self::SETTING_DATE_FORMAT), $data);
   

   Would be good to throw an exception if data is not numeric first.

3. +++ b/tests/src/Unit/Plugin/Tamper/TimeToDateTest.php
   @@ -0,0 +1,33 @@
   +    $plugin = new TimeToDate($config, 'timetodate', []);
   

   Can use $this->plugin here

Hi Ericgsmith,

Thanks for the feedback. Here is the updated patch and interdiff after working on the feedback.

Thanks
Vijay

This one is working well for me. How many do we need for an RTBC?

Thank you vijay.mayilsamy :)

I'd like to submit this patch to add the timetodate plugin against the latest alpha1 release