ordinal not shown for day in date with option limit-day-ordinals-to-…

…day-1 seboettg#172
glorieux-f · Sep 28, 2023 · 89e5a98 · 89e5a98
1 parent 024a956
commit 89e5a98
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/Rendering/Date/DatePart.php b/src/Rendering/Date/DatePart.php
@@ -197,16 +197,21 @@ protected function renderDay(DateTime $date)
 
         $text = $date->getDay();
         $form = !empty($this->form) ? $this->form : $this->parent->getForm();
+
         switch ($form) {
             case 'numeric':
                 break;
             case 'numeric-leading-zeros':
                 $text = sprintf("%02d", $text);
                 break;
             case 'ordinal':
-                $limitDayOrdinals =
+                // https://docs.citationstyles.org/en/v1.0.2/specification.html#locale-options
+                $day1only =
                     CiteProc::getContext()->getLocale()->filter("options", "limit-day-ordinals-to-day-1");
-                if (!$limitDayOrdinals || Layout::getNumberOfCitedItems() <= 1) {
+                if ($day1only) {
+                    if ((int)$text === 1) $text = Number::ordinal($text);
+                }
+                else if (Layout::getNumberOfCitedItems() <= 1) {
                     $text = Number::ordinal($text);
                 }
         }