Fix date_format modifier to keep the same behaviour as before

src/builtin/date_format.h

@@ -87,9 +87,12 @@ class DateFormatModifier : public Modifier
     {
         // parse the time
         ParsedTime parsed(datetime);
+
+        // If parsing was unsuccessful, throw so that the original string is kept
+        if (!parsed) throw false;
 
         // was this a success? if not we use the current time
-        return process(parsed ? parsed : time(nullptr), params);
+        return process(parsed, params);
     }
 
 

src/builtin/parsedtime.h

@@ -71,19 +71,19 @@ class ParsedTime
          */
         operator const timelib_tzinfo * () const { return _info; }
     };
-
-    /**
-     *  The actual time that was parsed
-     *  @var timelib_time
-     */
-    timelib_time *_time = nullptr;
 
     /**
      *  Errors that accured while parsing
      *  @var timelib_error_container
      */
     timelib_error_container *_errors = nullptr;
 
+    /**
+     *  The actual time that was parsed
+     *  @var timelib_time
+     */
+    timelib_time *_time = nullptr;
+
     /**
      *  Get the internal representation
      *  @return timelib_time *
@@ -95,9 +95,8 @@ class ParsedTime
      *  @return timelib_time *
      */
     operator const timelib_time * () const { return _time; }
-
-
-private:
+
+
     /**
      *  Constructor to get the current time
      *  @param  timezone        Timezone info