When the "Enable Publish Dates" is on AND the content's publish date is in the future and the content is in r

[chrisny454]

When I have this turned on....

And the content is scheduled to be in the future but is still in draft mode....

I get a 404 error and the review link does not work.

As soon as I change the date to be in the past the review link works again. Is there something I am doing wrong?
Fixing this functionality is crucial so that our staff can review content that is going to be published in the future otherwise the plugin is not going to meet our needs.

Let me know next steps, I am happy to test things out if you give me a place to start.

[edalzell]

@chrisny454 you're on the most recent version of this, v5.0?

[chrisny454]

@edalzell Yes I am on the most recent version - 5.0

[edalzell]

Hmm, I can't replicate this.

[edalzell]

@chrisny454 anything unusual in your set up? Does live preview work for you on that entry? If not, the issue has nothing to do w/ Review as we use LP to actually do the work

[chrisny454]

@edalzell So one thing that may be working against us is that our entry URLS are generated based on the date they were / are going to be published. For example our Collections URL definition:
/blog/{year}/{month}/{day}/{slug}

I dont think that matters to your plugin, but maybe?

BTW, Live preview DOES work for our future dated entries.

[chrisny454]

I believe I have a working solution for my needs and it may help someone else. I had to tweak your prepEntry() function first if statement like below:

 private function prepEntry(Entry $entry): Entry
    {
        if ($entry->collection()->dated() && $entry->date()->isFuture()) {
            if($entry->collection()->futureDateBehavior() === 'private' )
            {
                //when futureDateBehavior is set to private, Statamic denies access so an override is needed.
                $entry->collection()->futureDateBehavior('public');
            }
            if(strpos($entry->collection()->routes(),'{month}') || strpos($entry->collection()->routes(),'{day}') || strpos($entry->collection()->routes(),'{year}'))
            {
                //do not change the date because the uri is dynamically created by the entry date.
            }
            else
            {
                $entry->date(now());
            }            
        }
        //if the revision changes the slug, the url is wrong and the review link 404s
        $originalSlug = $entry->slug();
        if ($entry->hasWorkingCopy()) {
            $entry = $entry->fromWorkingCopy();
        }        
        $entry->slug($originalSlug);
        return $entry->published(true);
    }

[edalzell]

Thanks, we'll take a look at this shortly.