Closed (fixed)
Project:
Interval Field
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Feb 2013 at 12:58 UTC
Updated:
13 Jun 2014 at 01:27 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
rooby commentedRelated to this and also maybe easiest to do it at once is to split the interval_apply_interval() function so that there is a function that generates an interval string (used for the php formatter) and a function that applies applies an interval string to a date.
That way the interval_apply_interval() could be used with other fields that contain a php date nterval string.
For example, #1921038-2: Recurring interval checkout pane could provide a recurring payment action that takes an interval string and uses interval_apply_interval() (as it currently does).
This would allow the recurring payment interval to be taken from an interval field (or other field containing an interval string) via a token.
Or interval_apply_interval() could remain as is and have a new function for applying an interval string (or just use DateTime::modify directly).
It's late so I could just be dreaming out loud but I will review this again tomorrow.
Comment #2
larowlanIt uses the field api, so you can implement hook_field_formatter_info() or use Custom formatters.
Happy to accept patches but none that allow php entry by admins
Comment #3
rooby commentedYep, that's what I was planning to do.
There will be no entering php by anyone.
Patch will come tonight.
Comment #4
rooby commentedComment #5
rooby commentedPatch ahoy.
Comment #6
larowlanThanks rooby
Committed as c36264e