Incorrect Date Value

Sorry to ask again, just wanted to give this a nudge, please can someone help me out here, been pulling my hair out all afternoon! :(

I think what's happening is during your loop one of your dates is being calculated as:

new Date(2018, 11, 31);

Because there aren't 31 days in November javascript is switching the month to December which has 31 days, you can see this happening:

const test = new Date(2018, 11, 31);
console.log(test);

// Mon Dec 31 2018 00:00:00 GMT-0500 (Eastern Standard Time)

@REALRANDYALLEN - Hey realrandyallen,

Thanks so much for coming back to me - much appreciated! Yeah I agree, I think this is what's happening, any suggestions on how I can fix this?

Thank you again!

Sorry, further my previous post, November dates are unaffected. It's the 31st December which isn't being blocked out...

Check if the date is valid before adding it:

christmasDates = [
    @while($arrival <= $return)
        @if (date('n', $arrival) == 1)
            continue
        @endif
        
        @php
            $date = date('Y-m-d', strtotime('-1 month', $arrival));
            $check = DateTime::createFromFormat('Y-m-d', $date);
        
            if ($check && $check->format('Y-m-d') === $date) {
                print "new Date(" . date('Y', $arrival) . ", " . date('n', strtotime($date)) . ", " . date('d', $arrival) . "),";
            }

            $arrival = strtotime('+1 day', $arrival);
        @endphp
    @endwhile
];

Not the cleanest solution but it should work, I can't test it though

@REALRANDYALLEN - Thanks for coming back to me, unfortunately I'm getting the following error:

syntax error, unexpected 'if' (T_IF)

@DEVONIAN - I've amended my code above

@REALRANDYALLEN - Unfortunately that has been no different, damn code!! Thanks anyway.

Can anyone else help? This is a minor issue, any help would be much appreciated...

@DEVONIAN - What are your arrival and return dates in your env file? I’ll try to test this either tonight or in the morning

Not to do with the issue, but please see this big warning about using env() outside of config files: https://laravel.com/docs/5.7/configuration#configuration-caching

You won't be able to cache the config on your production box, which makes the app run faster. You should only use env() in a config file, and use the config() helper to get the values instead of env().

Use this package https://packagist.org/packages/cmixin/business-day + Illuminate\Suport\Carbon. No need to reinvent the wheel :) I've used it in a project doing something similar to what you're describing and it worked like a charm. Good luck!

@drfraker Except he's using js date objects, not php.

However, there are packages for momentjs that will easily create a date range like you can with carbon. Then you'd just need

const range = moment.range(new Date(2018, 11, 17), new Date(2012, 12, 10));

https://github.com/rotaready/moment-range