Unpublished repeaters items have 2048 instead of 2049 for status

[adrianbj]

Short description of the issue

As per the title

Expected behavior

I would expect that unpublished repeater items would also have StatusOn(1) and therefore be 2049.

Actual behavior

They end up with 2048

Optional: Suggestion for a possible fix

Not sure if this needs a fix because I don't really know if it's intended or not. It seems inconsistent to me, but I am guessing there is a valid reason, so maybe this is more of a question that an issue, but I am not sure.

[ryancramerdesign]

The two statuses are equivalent. In some cases, I'll use them for temporary runtime purposes but don't recall if I'm doing that here or not. In any case, there's no need to worry about the differing statuses, they should be inconsequential.

[adrianbj]

Unfortunately it doesn't seem to be inconsequential in this case - a repeater item with 2049 shows as "New", rather than unpublished. If you are creating an unpublished repeater via the API you need to make sure it is set to 2048 and not 2049 or it simply doesn't work as none of content of the repeater is shown.

[adrianbj]

@isellsoap - any chance of re-opening this please. I do think there is more to it than Ryan mentioned. I could be wrong of course, but I would really appreciate a little more clarification on my last point.

Perhaps @ryancramerdesign will still remember to get back to me, but I know how many Github notifications he must get and it must be hard to keep track of them all. Having it open again will be an unavoidable reminder :)
Thanks!

[ryancramerdesign]

Taking a closer look, it does look like FieldtypeRepeater uses the Page::statusOn for some identification. How are you creating your Repeater pages? Make sure you are using one of the built-in methods that are setup for this purpose, i.e.

$item = $page->repeater_field->makeNewItem();

or this

$field = $fields->get('repeater_field');
$item = $field->type->getBlankRepeaterPage($page, $field);

This is good to do not only for making sure it has the right status for repeaters, but also because it will use existing blank ready pages when available. If you are already using one of these and still seeing the wrong status, please let me know.

[adrianbj]

Yeah I noticed that use of statusOn. I guess I thought that the inconsistency was still a little weird.

I have put together a one click script that converts a PageTable field to a Repeater or Repeater Matrix field. It converts everything beautifully including all fields, templates, and of course the content of each of the item sub-pages. The thing is that I am not using makeNewItem because I am changing the parent of sub-page from the PageTable parent to the relevant Repeater parent (with some config changes), so I think in this case my hack to manually remove StatusOn is probably the best solution.

Anyway, I have a working solution and as long as you are happy, then I guess this isn't really a problem.

[adrianbj]

Sorry @ryancramerdesign - more thoughts on this - it currently seems impossible to use either of these on a repeater item, even if they were created via the admin GUI:

$item->addStatus(Page::statusUnpublished);
$item->removeStatus(Page::statusUnpublished);

Am I missing something?

[ryancramerdesign]

Adrian, check out the comments at the top of the FieldtypeRepeater.module file, it has a few notes in there about how Repeater pages use Page statuses and what the different combinations mean. I'm not positive this is the reason for what you are seeing, so you might also want to double check that the repeater items originated from an unformatted value (i.e. outputFormatting == false).