We have a scenario where users can create comments against news pages on a web-site.

The comments are created in code as children of the news page, and we require them to be saved as 'Ready to Publish', so an administrator can check them and subsequently publish them.

Following is the code we use to save the comments:

Note that comment submitters  may not be logged in, so we have a a default user (identified by AutomaticCreatorUserName) to save details as.

UnifiedPrincipal currentUser = UnifiedPrincipal.Current;

// Get our 'Auto create' user details from config

string username = SiteSettings.GetSettingString("AutomaticCreatorUserName");

string password = SiteSettings.GetSettingString("AutomaticCreatorPassword");

// Obtain the Iprincipal

IPrincipal principal = AuthenticationProvider.Authenticate(this, username, password);

// Set the CurrentUser to this one

PageBase.CurrentUser = principal as UnifiedPrincipal;

// Load the required page type

PageType commentPageType = PageType.Load(Names.PageTypeNewsComment);

// Create a new Comment page instance

PageData newPage = Global.EPDataFactory.GetDefaultPageData(CurrentPage.PageLink, commentPageType.ID, EPiServer.Security.AccessControlList.NoAccess);

// Set details for the page

newPage.PageName = "Comment created: " + DateTime.Now.ToShortDateString() + " " + DateTime.Now.ToShortTimeString();

newPage.VisibleInMenu = false;

((PropertyString)newPage.Property["CommentName"]).Value = txtCommentName.Text;

((PropertyString)newPage.Property["CommentEmail"]).Value = txtCommentEmail.Text;

((PropertyLongString)newPage.Property["CommentText"]).Value = HttpUtility.HtmlEncode(txtCommentText.Text);

// Now save the page

Global.EPDataFactory.Save(newPage, EPiServer.DataAccess.SaveAction.Commit, EPiServer.Security.AccessControlList.NoAccess);

// Restore previous user

PageBase.CurrentUser = currentUser;

Our problem is that when a user with sufficient permissions comes to publish the page, a new version of the page is created, and the original version still remains in the Task List. Do we need to use one of the 'Forcexxx' options of SaveAction instead of CheckIn?