From: Claudio Cambra <claudio.cambra@nextcloud.com>
Date: Tue, 8 Aug 2023 10:48:35 +0000 (+0800)
Subject: Ensure leading zero dates are also correctly parsed
X-Git-Tag: archive/raspbian/3.16.7-1_deb13u1+rpi1~1^2~92^2^2~7^2~8
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=7cfd4d877b88a252cd31c012b281abb8c1f1cf79;p=nextcloud-desktop.git

Ensure leading zero dates are also correctly parsed

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
---

diff --git a/src/gui/filedetails/datefieldbackend.cpp b/src/gui/filedetails/datefieldbackend.cpp
index 6c0436978..89556c8cc 100644
--- a/src/gui/filedetails/datefieldbackend.cpp
+++ b/src/gui/filedetails/datefieldbackend.cpp
@@ -25,16 +25,29 @@ namespace Quick
 DateFieldBackend::DateFieldBackend(QObject *const parent)
     : QObject(parent)
 {
+    _dateFormat = QLocale::system().dateFormat(QLocale::ShortFormat);
+
     // 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);
+    // pain to work with -- ensure instead we have the full, unambiguous year.
     // Check for specifically two y's, no more and no fewer, within format date
-    const QRegularExpression re("(?<!y)y{2}(?!y)");
+    const QRegularExpression yearRe("(?<!y)y{2}(?!y)");
 
-    if (auto match = re.match(_dateFormat); match.hasMatch()) {
+    // To prevent invalid parsings when the user submits a month with a leading
+    // zero, also add an alternative date format that checks with a leading zero
+    // This regex only matches, e.g. dd/M/yyyy which often is the default for
+    // short locale date formats, which removes the leading 0
+    const QRegularExpression monthRe("(?<!M)M{1}(?!M)");
+
+    if (const auto match = yearRe.match(_dateFormat); match.hasMatch()) {
         _dateFormat.replace(match.capturedStart(), match.capturedLength(), "yyyy");
     }
+
+    _leadingZeroMonthDateFormat = _dateFormat;
+
+    if (const auto match = monthRe.match(_dateFormat); match.hasMatch()) {
+        _leadingZeroMonthDateFormat.replace(match.capturedStart(), match.capturedLength(), "MM");
+    }
 }
 
 QDate DateFieldBackend::date() const
@@ -79,7 +92,12 @@ QString DateFieldBackend::dateString() const
 void DateFieldBackend::setDateString(const QString &dateString)
 {
     const auto locale = QLocale::system();
-    const auto date = locale.toDate(dateString, _dateFormat);
+    auto date = locale.toDate(dateString, _dateFormat);
+
+    if (!date.isValid()) {
+        date = locale.toDate(dateString, _leadingZeroMonthDateFormat);
+    }
+
     setDate(date);
 }
 
diff --git a/src/gui/filedetails/datefieldbackend.h b/src/gui/filedetails/datefieldbackend.h
index 5ed813c47..b44fc6822 100644
--- a/src/gui/filedetails/datefieldbackend.h
+++ b/src/gui/filedetails/datefieldbackend.h
@@ -83,6 +83,7 @@ private:
     QDate _maximumDate;
 
     QString _dateFormat;
+    QString _leadingZeroMonthDateFormat;
 };
 
 } // namespace Quick