Changeset 015ce46 in Klonkt

Timestamp:

06/21/2026 04:32:54 PM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

9effb80

Parents:

5a394e5

Message:

fix(inplannen): scheduled posts never went live + timezone bug

(1) Scheduler compared publish_at (ISO 'T..Z') string-wise with SQLite
CURRENT_TIMESTAMP ('YYYY-MM-DD HH:MM:SS'); because 'T' > space the condition
was never true → scheduled posts NEVER flipped to published. Now
datetime(publish_at) <= datetime('now') (both UTC, same format).
(2) datetime-local was stored as UTC without timezone → 18:10 CEST became
18:10 UTC (= 20:10 local). Now: client converts local input to UTC ISO on save,
pre-fills the field from UTC→local, and shows "Scheduled for" in readable local
time instead of raw UTC ISO.

Co-Authored-By: Claude <noreply@…>

Location:

src

Files:

• services/Scheduler.js (modified) (1 diff)
• services/i18n.js (modified) (3 diffs)
• views/pages/post-edit.ejs (modified) (2 diffs)

src/services/Scheduler.js

@@ -16 +16 @@
       SELECT p.id, p.title, p.content, u.username
       FROM posts p JOIN users u ON u.id = p.author_id
-      WHERE p.status = 'scheduled' AND p.publish_at IS NOT NULL AND p.publish_at <= CURRENT_TIMESTAMP
+      WHERE p.status = 'scheduled' AND p.publish_at IS NOT NULL AND datetime(p.publish_at) <= datetime('now')
     `).all();
     if (!due.length) return 0;

src/services/i18n.js

@@ -729 +729 @@
     'pedit.publish_at_label': 'Datum & tijd',
     'pedit.scheduled_for': 'Ingepland voor {d}',
+    'pedit.scheduled_prefix': 'Ingepland voor',
     'pedit.cancel': 'Annuleren',
     'pedit.publish': 'Publish',
@@ -1687 +1688 @@
     'pedit.publish_at_label': 'Date & time',
     'pedit.scheduled_for': 'Scheduled for {d}',
+    'pedit.scheduled_prefix': 'Scheduled for',
     'pedit.cancel': 'Cancel',
     'pedit.publish': 'Publish',
@@ -2645 +2647 @@
     'pedit.publish_at_label': 'Datum & Uhrzeit',
     'pedit.scheduled_for': 'Geplant für {d}',
+    'pedit.scheduled_prefix': 'Geplant für',
     'pedit.cancel': 'Abbrechen',
     'pedit.publish': 'Veröffentlichen',

src/views/pages/post-edit.ejs

@@ -215 +215 @@
           <div id="pe-sched-fields" style="margin-top:6px;<%= _scheduled ? '' : 'display:none' %>">
             <label style="display:block;font-size:12.5px;opacity:.8;margin-bottom:4px"><%= t('pedit.publish_at_label') %></label>
-            <input type="datetime-local" name="publish_at" <%= _scheduled ? '' : 'disabled' %> value="<%= post.publish_at ? String(post.publish_at).replace(' ','T').slice(0,16) : '' %>" style="padding:8px 10px;border-radius:8px;border:1px solid var(--rule,rgba(128,128,128,.4));background:transparent;color:inherit">
-            <% if (post.status === 'scheduled' && post.publish_at) { %><div style="font-size:12px;opacity:.7;margin-top:4px">⏳ <%= t('pedit.scheduled_for', { d: post.publish_at }) %></div><% } %>
+            <input type="datetime-local" id="pe-publish-at" name="publish_at" <%= _scheduled ? '' : 'disabled' %> data-iso="<%= post.publish_at || '' %>" value="" style="padding:8px 10px;border-radius:8px;border:1px solid var(--rule,rgba(128,128,128,.4));background:transparent;color:inherit">
+            <% if (post.status === 'scheduled' && post.publish_at) { %><div style="font-size:12px;opacity:.7;margin-top:4px"><span id="pe-sched-when" data-iso="<%= post.publish_at %>" data-label="<%= t('pedit.scheduled_prefix') %>">⏳ <%= t('pedit.scheduled_for', { d: post.publish_at }) %></span></div><% } %>
             <div style="font-size:11.5px;opacity:.65;margin-top:4px"><%= t('pedit.schedule_hint') %></div>
           </div>
@@ -224 +224 @@
               var box = document.getElementById('pe-sched-fields');
               if (!cb || !box) return;
-              var inp = box.querySelector('input[name="publish_at"]');
+              var inp = document.getElementById('pe-publish-at');
+              var pad = function (n) { return String(n).padStart(2, '0'); };
+              // Prefill: opgeslagen UTC → lokale datetime-local-waarde (zodat je je
+              // eigen tijdzone ziet, niet UTC).
+              if (inp && inp.dataset.iso) {
+                var d0 = new Date(inp.dataset.iso);
+                if (!isNaN(d0)) inp.value = d0.getFullYear() + '-' + pad(d0.getMonth() + 1) + '-' + pad(d0.getDate()) + 'T' + pad(d0.getHours()) + ':' + pad(d0.getMinutes());
+              }
+              // "Ingepland voor" in leesbare lokale tijd i.p.v. ruwe UTC-ISO.
+              var when = document.getElementById('pe-sched-when');
+              if (when && when.dataset.iso) {
+                var dw = new Date(when.dataset.iso);
+                if (!isNaN(dw)) when.textContent = '⏳ ' + when.dataset.label + ' ' + dw.toLocaleString();
+              }
               function sync() { box.style.display = cb.checked ? '' : 'none'; if (inp) inp.disabled = !cb.checked; }
               cb.addEventListener('change', sync); sync();
+              // Bij opslaan: lokale invoer → UTC-ISO via een hidden veld, zodat de
+              // server de juiste tijd opslaat (datetime-local heeft geen tijdzone).
+              var form = cb.closest('form');
+              if (form) {
+                form.addEventListener('submit', function () {
+                  if (inp) inp.removeAttribute('name');
+                  var old = form.querySelector('input[data-pa-utc]');
+                  if (old) old.remove();
+                  if (cb.checked && inp && inp.value) {
+                    var d2 = new Date(inp.value); // geïnterpreteerd in browser-tijdzone
+                    if (!isNaN(d2)) {
+                      var h = document.createElement('input');
+                      h.type = 'hidden'; h.name = 'publish_at'; h.setAttribute('data-pa-utc', '');
+                      h.value = d2.toISOString();
+                      form.appendChild(h);
+                    }
+                  }
+                });
+              }
             })();
           </script>