Merged
Conversation
added 6 commits
March 24, 2026 08:30
Reviewer's GuideReplaces SpoonDate-based Twig date/time formatting with IntlDateFormatter and introduces a custom time-ago implementation and localized labels, while adding a helper for parameterized labels. Sequence diagram for the new timeAgo Twig filtersequenceDiagram
participant TwigTemplate
participant FrontendTemplateModifiers as FrontendTemplateModifiers
participant FrontendLanguage as FrontendCoreLanguageLanguage
TwigTemplate->>FrontendTemplateModifiers: timeAgo(timestamp)
activate FrontendTemplateModifiers
FrontendTemplateModifiers->>FrontendTemplateModifiers: now = time()
FrontendTemplateModifiers->>FrontendTemplateModifiers: diff = |now - timestamp|
FrontendTemplateModifiers->>FrontendTemplateModifiers: compute seconds, minutes, hours, days, months, years
FrontendTemplateModifiers->>FrontendTemplateModifiers: select count and keySingular/keyPlural
alt count <= 0
FrontendTemplateModifiers->>FrontendLanguage: lbl(TimeAgoEmpty)
FrontendLanguage-->>FrontendTemplateModifiers: label
FrontendTemplateModifiers-->>TwigTemplate: label
else count == 1
FrontendTemplateModifiers->>FrontendLanguage: lbl(keySingular)
FrontendLanguage-->>FrontendTemplateModifiers: label
FrontendTemplateModifiers-->>TwigTemplate: label
else count > 1
FrontendTemplateModifiers->>FrontendLanguage: lblWithParameters(keyPlural, [count])
FrontendLanguage-->>FrontendTemplateModifiers: formattedLabel
FrontendTemplateModifiers-->>TwigTemplate: formattedLabel
end
deactivate FrontendTemplateModifiers
Class diagram for updated date and time Twig modifiersclassDiagram
class FrontendCoreEngineTemplateModifiers {
<<static>>
+string formatDate(var)
+string formatDateTime(var)
+string formatTime(var)
+string timeAgo(timestamp)
}
class BackendCoreEngineTemplateModifiers {
<<static>>
+string formatDate(var)
+string formatDateTime(var)
+string formatTime(var)
}
class CommonCoreTwigExtensionsBaseTwigModifiers {
<<static>>
+string spoonDate(timestamp, format, language)
}
class FrontendCoreLanguageLanguage {
<<static>>
+string lbl(key, fallback)
+string lblWithParameters(key, parameters, fallback)
+string getLabel(key, fallback)
}
class BackendCoreLanguageLanguage {
<<static>>
+string getInterfaceLanguage()
}
class FrontendCoreLanguageLocale {
<<static>>
+string frontendLanguage()
}
class IntlDateFormatter {
+IntlDateFormatter(locale, dateType, timeType, timezone, calendar, pattern)
+string format(timestamp)
}
class BackendCoreEngineAuthentication {
<<static>>
+User getUser()
}
class User {
+string getSetting(key)
}
%% Relationships
FrontendCoreEngineTemplateModifiers ..> FrontendCoreLanguageLocale : uses
FrontendCoreEngineTemplateModifiers ..> IntlDateFormatter : uses
FrontendCoreEngineTemplateModifiers ..> FrontendCoreLanguageLanguage : uses
BackendCoreEngineTemplateModifiers ..> BackendCoreLanguageLanguage : uses
BackendCoreEngineTemplateModifiers ..> IntlDateFormatter : uses
BackendCoreEngineTemplateModifiers ..> BackendCoreEngineAuthentication : uses
BackendCoreEngineAuthentication ..> User : returns
CommonCoreTwigExtensionsBaseTwigModifiers ..> BackendCoreLanguageLanguage : uses
CommonCoreTwigExtensionsBaseTwigModifiers ..> IntlDateFormatter : uses
FrontendCoreLanguageLanguage ..> FrontendCoreLanguageLanguage : calls getLabel
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey - I've found 4 issues, and left some high level feedback:
- The new IntlDateFormatter usages in the front- and backend TemplateModifiers replace user-/configurable date/time formats with hardcoded patterns (e.g. 'dd.MM.yyyy', 'EEEE d MMMM yyyy', 'HH:mm'), which changes behavior and ignores existing settings; consider deriving the pattern or style from the existing configuration instead of hardcoding.
- BaseTwigModifiers::spoonDate() now ignores its $format and $language parameters and always uses BackendLanguage::getInterfaceLanguage() with a fixed pattern, which breaks the previous contract and likely frontend usage; it should respect the passed format and language (or be clearly deprecated and replaced).
- The timeAgo() implementation no longer wraps the relative text in an with a title showing the exact date/time, changing the generated markup and losing the precise timestamp tooltip; if this is not intentional, consider preserving the previous HTML structure while still using the new labels.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The new IntlDateFormatter usages in the front- and backend TemplateModifiers replace user-/configurable date/time formats with hardcoded patterns (e.g. 'dd.MM.yyyy', 'EEEE d MMMM yyyy', 'HH:mm'), which changes behavior and ignores existing settings; consider deriving the pattern or style from the existing configuration instead of hardcoding.
- BaseTwigModifiers::spoonDate() now ignores its $format and $language parameters and always uses BackendLanguage::getInterfaceLanguage() with a fixed pattern, which breaks the previous contract and likely frontend usage; it should respect the passed format and language (or be clearly deprecated and replaced).
- The timeAgo() implementation no longer wraps the relative text in an <abbr> with a title showing the exact date/time, changing the generated markup and losing the precise timestamp tooltip; if this is not intentional, consider preserving the previous HTML structure while still using the new labels.
## Individual Comments
### Comment 1
<location path="src/Common/Core/Twig/Extensions/BaseTwigModifiers.php" line_range="193" />
<code_context>
+ public static function spoonDate($timestamp, $format = 'Y-m-d H:i:s', $language = 'en')
</code_context>
<issue_to_address>
**issue (bug_risk):** spoonDate no longer respects its $format and $language parameters and always formats a date only.
Previously this delegated to `\SpoonDate::getDate($format, $timestamp, $language)`, which respected the provided format, language, and optional time component. The new version hardcodes `BackendLanguage::getInterfaceLanguage()`, ignores `$language` and `$format`, and always uses `'dd MMMM yyyy'`, changing behavior and potentially breaking existing templates relying on custom formats or date+time output. Please either (a) translate `$format` into an `IntlDateFormatter` pattern and use `$language` as the locale (falling back to styles when no format is given), or (b) keep `spoonDate` as a thin wrapper around the legacy behavior and introduce a separate Intl-based helper.
</issue_to_address>
### Comment 2
<location path="src/Frontend/Core/Engine/TemplateModifiers.php" line_range="36-42" />
<code_context>
- // format the date
- return SpoonDate::getDate($format, (int) $var, BackendLanguage::getInterfaceLanguage());
+ $date = new IntlDateFormatter(
+ BackendLanguage::getInterfaceLanguage(),
+ IntlDateFormatter::SHORT,
</code_context>
<issue_to_address>
**issue (bug_risk):** formatDate now ignores the configured frontend date format and uses a hardcoded pattern.
The previous implementation respected `Core.date_format_short` from `fork.settings`, enabling per-site date format configuration. The new hardcoded `'dd.MM.yyyy'` pattern changes existing behavior and ignores that CMS setting. If you still want to use Intl while honoring configuration, consider deriving the Intl pattern from the stored format or storing Intl-compatible patterns in settings. Otherwise, sites using custom date formats may see unexpected changes.
</issue_to_address>
### Comment 3
<location path="src/Backend/Core/Engine/TemplateModifiers.php" line_range="30-36" />
<code_context>
- // format the date
- return SpoonDate::getDate($format, (int) $var, BackendLanguage::getInterfaceLanguage());
+ $date = new IntlDateFormatter(
+ BackendLanguage::getInterfaceLanguage(),
+ IntlDateFormatter::SHORT,
+ IntlDateFormatter::NONE,
+ null,
+ null,
+ 'dd MMMM yyyy'
+ )->format((int) $var);
+
</code_context>
<issue_to_address>
**issue (bug_risk):** Backend formatDate no longer respects per-user date format settings.
Previously this used `Authentication::getUser()->getSetting('date_format')`, so each user could control their own date format. The new code hardcodes `'dd MMMM yyyy'`, removing that per-user configurability. Please consider deriving the pattern from the user’s setting (e.g. storing Intl-compatible patterns or mapping legacy formats to Intl) instead of hardcoding it.
</issue_to_address>
### Comment 4
<location path="src/Frontend/Core/Engine/TemplateModifiers.php" line_range="250-254" />
<code_context>
+ return Language::lbl('TimeAgoEmpty');
+ }
+
+ if ($count === 1) {
+ return Language::lbl($keySingular);
+ }
+
+ return Language::lblWithParameters(
+ $keyPlural,
+ [
</code_context>
<issue_to_address>
**question:** timeAgo now returns plain text without the previous <abbr> wrapper or exact timestamp tooltip.
Previously, the relative time was wrapped in an `<abbr>` with a `title` showing the exact timestamp, aiding accessibility and precision on hover. The new version returns only the localized relative string. If this change wasn’t intentional, consider keeping the `<abbr>` wrapper (with the precise timestamp in `title`) while still using the new `Language` labels for the visible text.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| public static function formatTime($var): string | ||
| { | ||
| // get setting | ||
| $format = Authentication::getUser()->getSetting('time_format'); |
Member
There was a problem hiding this comment.
Niet logischer om dit te verwijderen. Dan is er geen referentie meer naar Spoon.
Denk dat je wel alle templates snel kan doorzoeken om te zien of het wordt gebruikt.
Member
There was a problem hiding this comment.
Al denk ik dat het wel nuttig is om een method aan te bieden waarmee we het formaat kunnen meegeven. Niet?
Author
There was a problem hiding this comment.
Ik heb de naam veranderd van de method in de base twig modifier, zodat die wel nog kan gebruikt worden met format als parameter. Want die werd idd op verschillende plaatsen nog gebruikt, en dan kunnen we die gebruiken om format mee te geven 4d7a140
| public static function formatTime($var): string | ||
| { | ||
| // get setting | ||
| $format = FrontendModel::get('fork.settings')->get('Core', 'time_format'); |
Author
There was a problem hiding this comment.
Nee die zou ik er nog niet uithalen want die settings word in formbuilder ook gebruikt.
added 6 commits
March 24, 2026 09:11
…aGridFunctions where they were still used.
…format to filter. fix twig-cs warnings on templates
…t was still used + fix index twig template for twig cs fixer
| $format = Authentication::getUser()->getSetting('datetime_format'); | ||
| $format = 'j F Y H:i'; | ||
|
|
||
| // format the date according the user his settings |
Member
There was a problem hiding this comment.
Zou de comment dan ook weghalen.
| $format = 'j F Y H:i'; | ||
|
|
||
| // format the date according the user his settings | ||
| return SpoonDate::getDate($format, $timestamp, BackendLanguage::getInterfaceLanguage()); |
Member
There was a problem hiding this comment.
Eventueel ook al SpoonDate weghalen. Als je dat in apart ticket gaat doen, mag je dit negeren.
| $format = Authentication::getUser()->getSetting('time_format'); | ||
| $format = 'H:i'; | ||
|
|
||
| // format the date according the user his settings |
| $format = 'H:i'; | ||
|
|
||
| // format the date according the user his settings | ||
| return SpoonDate::getDate($format, $timestamp, BackendLanguage::getInterfaceLanguage()); |
Member
There was a problem hiding this comment.
Eventueel ook al SpoonDate weghalen. Als je dat in apart ticket gaat doen, mag je dit negeren.
Author
There was a problem hiding this comment.
Ja ik heb een aparte taak aangemaakt daarvoor
| @@ -129,7 +129,7 @@ public static function getTime(int $timestamp): string | |||
| public static function getTimeAgo(int $timestamp): string | |||
| { | |||
| // get user setting for long dates | |||
| <h3> | ||
| {% if archive.month %} | ||
| {{ archive.start_date|spoondate('F Y',LANGUAGE) }} | ||
| {{ archive.start_date|customformatdate('EEEE d MMMM yyyy', LANGUAGE) }} |
Member
There was a problem hiding this comment.
Dat formaat klopt toch niet?
F Y → Maart 2026
EEEE d MMMM yyyy → Donderdag 26 maart 2026
best overal eens goed nakijken?
Author
There was a problem hiding this comment.
1bec8cf heb nog eens nagekeken maar is enkel in deze file
jonasdekeukelaere
approved these changes
Apr 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type
Pull request description
Replaces SpoonDate with IntlDateFormatter to format dates and timestamps.
Summary by Sourcery
Replace legacy date utilities with IntlDateFormatter-based Twig filters for consistent date, time, and relative time formatting across frontend, backend, and shared templates.
Enhancements:
Documentation: