Comments (3)
Hi Cody,
I'm doing great thank you! How about you?
I didn't run into any issues with Recurr for now so thank you for pointing out the lack of support with this library. There are indeed some years old issues like #137 that were never solved although a new release was pushed a few weeks ago.
The php-rrule library on the other hand looks promising and is actively maintained. Also if it can help us implementing the pagination feature we were talking about then I am for sure not opposed to switching to it as long as we can assure backwards compatibility with the current version.
I look forward for your PR!
Thanks again for your support :)
from acf-rrule.
Hi Marc
I have created a patch to include php-rule as suggested in this post. However, at closer look, it does not necessarily seem to be a wise choice.
The main issue is that the RRule
object is "immutable". I have partly solved this by creating a deriving class, that allows values to be set after instantiation of the object (as it is used in "update_value"). However, this is not complete, as currently the __toString()
-function relies on the values that are stored in the $rule
-property array, rather than the properties that are set by the setters at the moment. So the setters would have to set the property in the rule as well.
Then there are limitations with php-rule as well - or maybe with the implementation of this field in general: while there is sort of support for DTSTART
, there is no such support of EXDATE
or RDATE
which both influence the produced series of dates.
So what I am looking for is a way to implement more into the ACF field. I was wondering if there is interest from your side on this as well.
If yes, I would create a new issue for the discussion of that. If no, I'd probably start a fork and we can always merge together later.
Thank you for your work and time!
Bhjuagenda
from acf-rrule.
Hi Bhujagendra,
Thanks again for your work on this.
I don't think we need support for EXDATE
or RDATE
. There is no support in simshaun/recurr as well.
There are other ways to manage exceptions like creating a separate repeater field with date fields so you can exclude them later in a custom function. Or you can use the RRule field inside of a repeater field so you can create multiple periods.
That being said it would be great if we could manage them directly inside the RRule field.
I'm personally happy with the current implementation. Switching to php-rrule was meant to solve the RFC compliance issue and I don't think we absolutely need that as long as we are able to convert a single string to a series of dates in the end.
For now as far as I can see there is no advantage to switching to php-rrule as it does not solve the RFC compliance issue (we would have to take off the DTSTART
attribute from the string) and it seems to bring a bit more complexity to the current implementation.
Of course I would be happy to take your contributions if you want to bring new features to the plugin. Feel free to open a new discussion ;)
Thank you!
from acf-rrule.
Related Issues (20)
- Additional Validation Needed HOT 3
- [i18n] Textdomain doesn't match the plugin slug
- Option for Monthly Recurrence on Multiple Days of the Week HOT 9
- On save error HOT 5
- Can you add a feature to disable the required fields HOT 9
- UI doesn't reflect saved value HOT 5
- end type: count, no end date? HOT 2
- Arrays returned as string HOT 2
- Selected Month Days aren't reflected in the calendar HOT 4
- start_time field not working HOT 3
- End date becomes one day earlier after saving post. HOT 5
- Critical error when date fields are set but weekdays are not selected on drafts HOT 7
- Issue with DTEND HOT 2
- Repeat events on front end HOT 5
- Field not showing after 1.4 update HOT 3
- Timezone support
- Javscript Error when attaching to a Gutenberg Custom Post Type HOT 2
- Plugin dependencies
- Support for DTEND attribute
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from acf-rrule.