PageField custom PHP code returning blank

[a-ok]

Hi folks,

I want to return a few repeater items via a PageField and originally I would do this by writing the commented out code in the PHP area provided BUT now it states I should be doing this in ready.php via a hook.

I have written the below... following from the example given in the backend but it's not returning anything in the PageField dropdown. The message I set up to do a bit of debugging seems to be returning the correct items.

Any thoughts? And am I able to return the title of the repeater item being chosen as the getForPage()->title as shown below (how I previously did it).

$wire->addHookAfter('InputfieldPage::getSelectablePages', function($event) {
	if ($event->object->name == 'home_whatson_article') {

		$event->return = $event->pages->find("template=repeater_events_detail_dates, events_detail_dates_start_date>today, sort=events_detail_dates_start_date");

		$this->message($event->return);

		/*
		$options = new PageArray();
		foreach ($pages->find("template=repeater_events_detail_dates, events_detail_dates_start_date>today, sort=events_detail_dates_start_date") as $e) {
			$e->title = $e->getForPage()->title;
			$options->add($e);
		}
		return $options;
		*/


	}
});

 

[abdus]

InputfieldPage::getSelectablePages doesnt work with all input types

	protected static $defaultInputfieldClasses = array(
		'InputfieldSelect', // works
		'InputfieldSelectMultiple', // works
		'InputfieldCheckboxes', // works
		'InputfieldRadios',  // works
		'InputfieldAsmSelect', // works
		'InputfieldPageListSelect',  // doesnt work
		'InputfieldPageAutocomplete', // doesnt work
		);

Because it's called only if inputfield has addOption() method

// InputfieldPage.module
/**
 * Get delegate Inputfield for page selection
 * 
 * @return Inputfield|null
 * @throws WireException
 * 
 */
public function getInputfield() {
    // ...
    if(method_exists($inputfield, 'addOption')) {
        
        $children = $this->getSelectablePages($page);
        
        if($children) {
            foreach($children as $child) {
                $label = $this->getPageLabel($child);
                $inputfield->addOption($child->id, $label);
            }
        }
        
    } 
    // ...
}

 

[a-ok]

1 minute ago, abdus said:

InputfieldPage::getSelectablePages doesnt work with all input types

	protected static $defaultInputfieldClasses = array(
		'InputfieldSelect', // works
		'InputfieldSelectMultiple', // works
		'InputfieldCheckboxes', // works
		'InputfieldRadios',  // works
		'InputfieldAsmSelect', // works
		'InputfieldPageListSelect',  // doesnt work
		'InputfieldPageAutocomplete', // doesnt work
		);

Because it's called only if inputfield has addOption method()

// InputfieldPage.module
/**
 * Get delegate Inputfield for page selection
 * 
 * @return Inputfield|null
 * @throws WireException
 * 
 */
public function getInputfield() {
    // ...
    if(method_exists($inputfield, 'addOption')) {
        
        $children = $this->getSelectablePages($page);
        
        if($children) {
            foreach($children as $child) {
                $label = $this->getPageLabel($child);
                $inputfield->addOption($child->id, $label);
            }
        }
        
    } 
    // ...
}

 

Thanks, abdus. My PageReference field is InputfieldAsmSelect

[abdus]

I tried using asmSelect, it works fine on my setup. Maybe you need include=all (repeater pages are hidden by default)?

$wire->addHookAfter('InputfieldPage::getSelectablePages', function($event) {
    if($event->object->hasField == 'tags') {
        $event->return = $event->pages->find('template=repeater_seo, include=all');
    }
});

[a-ok]

4 minutes ago, abdus said:

I tried using asmSelect, it works fine on my setup. Maybe you need include=all (repeater pages are hidden by default)?

$wire->addHookAfter('InputfieldPage::getSelectablePages', function($event) {
    if($event->object->hasField == 'tags') {
        $event->return = $event->pages->find('template=repeater_seo, include=all');
    }
});

I even changed it just to find all pages with a specific template (not a repeater) AND to a simple Select field (not ASM) and I get nothing.

My PageReference field is in a repeater... could this be something that is killing it?

[a-ok]

Looks like it's to do with it being in a repeater

 

[abdus]

14 minutes ago, abdus said:

Maybe you need include=all (repeater pages are hidden by default)?

 

[a-ok]

2 minutes ago, abdus said:

 

Sorry it's the same field... for testing purposes I just tried to find events-detail templates. If you stick your example in a repeater it won't work.

$event->return = $event->pages->find("template=events-detail");

 

[a-ok]

It has to do with the PageField being inside a repeater... it works fine outside it.

[abdus]

It works fine inside repeaters too

$wire->addHookAfter('InputfieldPage::getSelectablePages', function($event) {
    if($event->object->hasField == 'pageRef') {
        $event->return = $event->pages->find('template=repeater_testRepeater, include=all');
    }
});

 

[a-ok]

2 minutes ago, abdus said:

It works fine inside repeaters too

$wire->addHookAfter('InputfieldPage::getSelectablePages', function($event) {
    if($event->object->hasField == 'pageRef') {
        $event->return = $event->pages->find('template=repeater_testRepeater, include=all');
    }
});

 

Thanks for testing, abdus.

It turns out my line if ($event->object->name == 'home_whatson_article') { needed to be your line if ($event->object->hasField == 'home_whatson_article') {

[abdus]

Ah, ok I was directly copying from the field settings. It's filled with correct field names and everything

[a-ok]

54 minutes ago, abdus said:

Ah, ok I was directly copying from the field settings. It's filled with correct field names and everything

So was I! Which version are you on?

 

[abdus]

Latest. v3.0.75

[Robin S]

3 hours ago, oma said:

It turns out my line if ($event->object->name == 'home_whatson_article') { needed to be your line if ($event->object->hasField == 'home_whatson_article') {

Ryan committed a change back in May in response to this issue but it hasn't been merged to the master branch yet.

[a-ok]

13 hours ago, Robin S said:

Ryan committed a change back in May in response to this issue but it hasn't been merged to the master branch yet.

Thanks as always, Robin. That's great.