From 48c49fca60da46c8066e7e8dc8e202b055ff665b Mon Sep 17 00:00:00 2001
From: Claudio Cambra <claudio.cambra@nextcloud.com>
Date: Tue, 8 Aug 2023 17:27:11 +0800
Subject: [PATCH] Customise locale default date format to ensure we eliminate
 issues parsing two-digit year strings

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
---
 src/gui/filedetails/datefieldbackend.cpp | 21 ++++++++++++++++++---
 src/gui/filedetails/datefieldbackend.h   |  4 +++-
 2 files changed, 21 insertions(+), 4 deletions(-)

diff --git a/src/gui/filedetails/datefieldbackend.cpp b/src/gui/filedetails/datefieldbackend.cpp
index 0764ac2e1..8377afad9 100644
--- a/src/gui/filedetails/datefieldbackend.cpp
+++ b/src/gui/filedetails/datefieldbackend.cpp
@@ -15,12 +15,28 @@
 #include "datefieldbackend.h"
 
 #include <QLocale>
+#include <QRegularExpression>
 
 namespace OCC
 {
 namespace Quick
 {
 
+DateFieldBackend::DateFieldBackend(QObject *const parent)
+    : QObject(parent)
+{
+    // Ensure the date format is for a full year. QLocale::ShortFormat often
+    // provides a short year format that is only two years, which is an absolute
+    // pain to work with -- ensure instead we have the full, unambiguous year
+    _dateFormat = QLocale::system().dateFormat(QLocale::ShortFormat);
+    // Check for specifically two y's, no more and no fewer, within format date
+    const QRegularExpression re("(?<!y)y{2}(?!y)");
+
+    if (auto match = re.match(_dateFormat); match.hasMatch()) {
+        _dateFormat.replace(match.capturedStart(), match.capturedLength(), "yyyy");
+    }
+}
+
 QDateTime DateFieldBackend::dateTime() const
 {
     return _dateTime;
@@ -53,14 +69,13 @@ void DateFieldBackend::setDateTimeMsecs(const qint64 dateTimeMsecs)
 
 QString DateFieldBackend::dateTimeString() const
 {
-    const auto locale = QLocale::system();
-    return _dateTime.toString(locale.dateFormat(QLocale::ShortFormat));
+    return _dateTime.toString(_dateFormat);
 }
 
 void DateFieldBackend::setDateTimeString(const QString &dateTimeString)
 {
     const auto locale = QLocale::system();
-    const auto dt = locale.toDateTime(dateTimeString, locale.dateFormat(QLocale::ShortFormat));
+    const auto dt = locale.toDateTime(dateTimeString, _dateFormat);
     setDateTime(dt);
 }
 
diff --git a/src/gui/filedetails/datefieldbackend.h b/src/gui/filedetails/datefieldbackend.h
index dde64187f..d0df944ef 100644
--- a/src/gui/filedetails/datefieldbackend.h
+++ b/src/gui/filedetails/datefieldbackend.h
@@ -39,7 +39,7 @@ class DateFieldBackend : public QObject
     Q_PROPERTY(bool validDateTime READ validDateTime NOTIFY dateTimeChanged NOTIFY minimumDateTimeChanged NOTIFY maximumDateTimeChanged)
 
 public:
-    explicit DateFieldBackend() = default;
+    explicit DateFieldBackend(QObject *const parent = nullptr);
 
     [[nodiscard]] QDateTime dateTime() const;
     [[nodiscard]] qint64 dateTimeMsecs() const;
@@ -73,6 +73,8 @@ private:
     QDateTime _dateTime = QDateTime::currentDateTimeUtc();
     QDateTime _minimumDateTime;
     QDateTime _maximumDateTime;
+
+    QString _dateFormat;
 };
 
 } // namespace Quick
-- 
2.30.2