Custom Page Class: Creating Pages with new Page() Not Visible in Admin Interface

[klor]

Short description of the issue

When using the createPage method in a custom class that extends Page, the instantiation of the Page object using $p = new Page(); is not working as expected, even though it should. The issue is that the created page is not visible in the ProcessWire admin interface, although it is stored in the database.

<?php namespace ProcessWire;

class ProductPage extends Page {

	public function createPage($array) {
		$pages = $this->wire('pages');

		$p = new Page();
		$p->setOutputFormatting(false);
		$p->template = 'my-template'; // valid template
		$p->parent = $pages->get('/foo/'); // parent exists, template family allows 'my-template'
		$p->name = 'bar';
		$p->title = 'bar';
		$p->save();
	}
}

Expected behavior

The createPage method should successfully create a new page with the specified template, parent, name, and title, and it should be visible in the ProcessWire admin interface.

Actual behavior

1. The created page is not visible in the ProcessWire admin interface, even though it is stored in the database.
2. The code works when executed from '/site/templates/my-template.php' (a page is created).

Optional: Screenshots/Links that demonstrate the issue

N/A

Optional: Suggestion for a possible fix

Your possible fix goes here.

Steps to reproduce the issue

1. Create a custom class that extends Page with a createPage method, similar to the code provided in the issue description.
2. Instantiate the custom class and call the createPage method.
3. Check the ProcessWire admin interface to see that the created page is not visible.

Setup/Environment

• ProcessWire version: 3.0.229
• PHP version: 8.2
• MySQL version: 10.6.13-MariaDB
• 3rd party modules: TracyDebugger, ProcessWireUpgrade, ProcessDatabaseBackups, ImportPagesCSV

[BernhardBaumrock]

I can reproduce this. I didn't find the issue on my quick research.

Not a solution, but a FYI. This works:

    $p = $pages->new([
      'template' => 'basic-page',
      'parent' => $pages->get("/"),
      'name' => 'foo',
      'title' => 'foo',
    ]);

[ryancramerdesign]

@klor @BernhardBaumrock Since you are doing this from within a class that extends Page, it has direct access to something like $p->template, being a protected property. So you can't set it to a string, you'd have to set it as a literal Template object like $p->template = wire()->templates->get('basic-page'). If you want to use a string, you'll have use $p->setTemplate('basic-page');. If you aren't inside a Page class, then the code would work fine as-is.

[matjazpotocnik]

@klor, are you ok with Ryan's answer? If so, could you close this issue?

[klor]

Thanks for providing background info, @ryancramerdesign - and kudos to you @matjazpotocnik for helping Ryan and make things run smoothly.