It would be great if we could apply the Date Popup module as dependency in the revisioning scheduler,
in order to allow the administrator to choose whether (s)he wants to complete the schedule fields as text
or as a date popup.

Thanks,
Alex

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexkont’s picture

Status: Active » Needs review
FileSize
6.78 KB

The part patch checks only if the date popup module is enabled.
I should create an admin page, too to make the configuration easier for the administrator .

Thanks,
Alex

RdeBoer’s picture

Thanks Alex!
Rik

rv0’s picture

Works, but the validation makes it impossible to publish something right away when clicking the publish link. (probably, because "now" is in the past)

JesseDP’s picture

@rv0
It seems that date_popup has a problem with the default_value.

Overhere, it seems to work if I make the default_value for revisioning_scheduler_date and revisioning_scheduler_time date('Y-m-d H:i');

gmclelland’s picture

FYI... The https://drupal.org/project/revision_scheduler module has a similar issue with the date_popup and default value at https://drupal.org/node/1342828#comment-6740076

The patch that is suggested has the same time format as what is suggested in #4

vellines’s picture

Attached patch is a rewrite of @alexkont's from #1 written against latest in 7.x-1.x-dev. It includes a config setting to enable date popup widget only if the date_popup module is enabled.

Status: Needs review » Needs work

The last submitted patch, 6: revision_scheduler-adds_date_popup_option-2007134-6.patch, failed testing.

vellines’s picture

Version: 7.x-1.5 » 7.x-1.x-dev

Changing target version for this issue to -dev since new features are typically implemented there rather than a specific branch.

The last submitted patch, 6: revision_scheduler-adds_date_popup_option-2007134-6.patch, failed testing.

Olarin’s picture

Re-formatting patch from #6 and making checks for popup setting more consistent.

vipul tulse’s picture

Assigned: Unassigned » vipul tulse