Minor API update for when saving a new page

[ryan]

This is relatively minor, but I realized a lot of us were repeatedly using the same code to populate a required 'name' field when creating new pages via the API. As a result, I figured I'd build it in. Here's the commit note:

Update $pages->save() so that it auto-generates a $page->name field when none is provided in the new page. It generates it from the 'title' field. This makes for more convenient API usage, especially when importing lots of pages. It also accounts for duplicate name fields and adds an incrementing number to the end when it finds duplicates.

Previously if you tried to save a page without a 'name' field it would throw an exception. Now it won't, as long as the page has a title. This should not affect any existing code. But it does mean that future page imports may be a little simpler since you don't have to consider the 'name' field.

[Frank Vèssia]

Ryan, i tried with latest commit but i get a page name "0". No exception but name of the page is not the title...

simple code:

<?php
$s = new Page();
$s->template 	= $templates->get("submit");
$s->parent	= $pages->get($userid);
$s->title     = "Submissions";
$s->save();

Maybe because my parent page is an user?  

[Adam Kiss]

Sorry for hijacking this post a little, but you can add page to user record???

[Frank Vèssia]

@adamkiss: sure...

[ryan]

Sevarf2, I can't seem to duplicate here. I started from a new install and pasted this into a template:

<?php
$p = new Page();
$p->template = $templates->get('basic-page');
$p->parent = $pages->get($user->id);
$p->title = 'Submissions';
$p->save();

Then ended up with this page:

/processwire/access/users/admin/submissions/

Can you think of any other factors or other modules that might be running? I'm also curious about the page name "0". I just committed an update that may fix this in your case–can you grab the latest commit and let me know if that fixes it?

Thanks,

Ryan

[Frank Vèssia]

Tried now with latest commit. It works good now...