Laravel Carbon Data Missing

75,027

Solution 1

tl;dr

Your date string and your date format is different, you have to change the format string or modify the date string so they match.

Explanation

The Problem

This error arises when Carbon's createFromFormat function receieves a date string that doesn't match the passed format string. More precisely this comes from the DateTime::createFromFormat function, because Carbon just calls that:

public static function createFromFormat($format, $time, $tz = null)
{
    if ($tz !== null) {
        $dt = parent::createFromFormat($format, $time, static::safeCreateDateTimeZone($tz));
    } else {
        $dt = parent::createFromFormat($format, $time); // Where the error happens.
    }

    if ($dt instanceof DateTime) {
        return static::instance($dt);
    }

    $errors = static::getLastErrors();
    throw new InvalidArgumentException(implode(PHP_EOL, $errors['errors'])); // Where the exception was thrown.
}

Not enough data

If your date string is "shorter" than the format string like in this case:

Carbon::createFromFormat('Y-m-d H:i:s', '2017-01-04 00:52');

Carbon will throw:

InvalidArgumentException in Carbon.php line 425:
Data missing

Too much data

If your date string is "longer" than the format string like in this case:

 Carbon::createFromFormat('Y-m-d H:i', '2017-01-02 00:27:00');

Carbon will throw:

InvalidArgumentException in Carbon.php line 425:
Trailing data

Under the hood

According to the documentation on mutators the default date format is: 'Y-m-d H:i:s'. The date processing happens in the Model's asDateTime function. In the last condition the getDateFormat function is called, thats where the custom format comes from. The default format is defined in the Database's Grammar class.

Solution

You have to make sure that the date string matches the format string.

Change the format string

You can override the default format string like this:

class Event extends Model {
    protected $dateFormat = 'Y-m-d H:i';
}

There is two problem with this approach:

  • This will apply to every field defined in the model's $dates array.
  • You have to store the data in this format in the database.

Edit and format the date strings

My recommended solution is that the date format should stay the default 'Y-m-d H:i:s' and you should complete the missing parts of the date, like this:

public function store(Request $request) {
    $requestData = $request->all();
    $requestData['start_time'] .= ':00';
    $requestData['end_time'] .= ':00';
    $event = new Event($requestData);
    $event->save();
}

And when you want to use the date you should format it:

public function show(Request request, $eventId) {
    $event = Event::findOrFail($eventId);
    $startTime = $event->start_time->format('Y-m-d H:i');
    $endTime = $event->end_time->format('Y-m-d H:i');
}

Of course the fields should be mutated to dates:

class Event extends Model {
    protected $dates = [
        'start_time',
        'end_time',
        'created_at',
        'updated_at',
        'deleted_at',
    ];
}

Solution 2

Models

This function disabled, the emulations for carbon in Datetimes https://laravel.com/docs/5.0/eloquent#date-mutators

public function getDates()
{
    return [];
}

Solution 3

You can set the $dateFormat in your model as Christian says, but if you don't want to imply the updated_at and created_at fields into the operation you can use events to "correct" the datetime object before saving it into the database.

Here you have the official doc about it: https://laravel.com/docs/5.2/eloquent#events

Solution 4

You need to set protected $dateFormat to 'Y-m-d H:i' in your model, see https://laravel.com/docs/5.2/eloquent-mutators#date-mutators

Share:
75,027
Hardist
Author by

Hardist

Updated on January 21, 2020

Comments

  • Hardist
    Hardist over 4 years

    In my model I have the following:

    protected $dates = [
        'start',
        'end',
        'created_at',
        'updated_at'
    ];
    

    I am using a datetime picker to insert the start and end dates, in this format:

    2016-01-23 22:00
    

    Without the seconds. When I do it like this, I get this error:

    InvalidArgumentException in Carbon.php line 425:
    Data missing
    at Carbon::createFromFormat('Y-m-d H:i:s', '2016-01-23 22:00') in Model.php line 3015
    

    If I do include the seconds, it works. The seconds are not important to me, and I do not want to include them in my datetime picker fields. Any way around this so I can still use those fields as date fields?

  • Hardist
    Hardist over 8 years
    Accepting this as it gives me more options, totally missed the $dateFormat option.
  • Bhargav Nanekalva
    Bhargav Nanekalva about 7 years
    Great answer. Such a quality answer. I couldn't help but comment this unproductive comment. :)
  • Admin
    Admin about 6 years
    I have the same problem this solution hasnt helped me,can you please take a look
  • Jonas D.
    Jonas D. over 5 years
    This is great for datetimes, but a PITA for date-only fields. It doesn't make sense to add a time to those & they also can't be stored that way; however there's a big ambiguity in naming, as they are "dates" in the truest sense
  • mytharcher
    mytharcher almost 5 years
    When I am using Postgresql, some time values are saved without millisecond part (when it is just zero). Then the date format will always be making error like above. So at last I can only fix it with specific accessor for the field. But this solution just ignored the original data format.
  • Karsai Péter
    Karsai Péter over 4 years
    The answer is perfect, but IMO, according to Laravel's documentation, this shouldn't be required: When a column is considered a date, you may set its value to a UNIX timestamp, date string (Y-m-d), date-time string, or a DateTime / Carbon instance. The date's value will be correctly converted and stored in your database See here. The reality is that it doesn't work according to the docs, and we must resort to hacks like this.