SaaS CMS has officially launched! Learn more now.

Jan 25, 2010
(1 votes)

Generic overloads of GetPage and GetChildren - To be or not to be?

Joel Abrahamsson put some light on some issues with the newly added generic overloads GetPage and GetChildren on DataFactory as well as the generic overload of ShallowCopy on PageData. These APIs were added mainly to give developers a way to hook in their own PageData implementation in a simple way which was used by the PageTypeTool and was mentioned by Daniel Rodin in his blog post Working with your own PageData types in CMS5 R2 SP2. However, as the use of these methods have been somewhat outdated by the introduction of PageTypeBuilder and as we agree with Joel’s feedback, we are now considering a removal of these APIs by the release of CMS 6.

Our only remaining concern is backward compatibility which is something we struggle to maintain as much as possible (even if we are shipping a new major version).  However our qualified guess (after all we can only guess) is that the adoption of these methods are not that widely spread. If we also take into account that we can create extension methods that replaces all but one of the APIs planned to be cut we think it should be rather safe to get rid of the methods in CMS 6. If you disagree, continue reading.

The signatures that are at stake:

On EPiServer.DataFactory:

public IEnumerable<T> GetChildren<T>(PageReference pageLink)
where T: PageData, new();

public IEnumerable<T> GetChildren<T>(PageReference pageLink, ILanguageSelector selector)
where T: PageData, new();

public T GetPage<T>(PageReference pageLink)
where T: PageData, new();

public T GetPage<T>(PageReference pageLink, ILanguageSelector selector)
where T: PageData, new();

On EPiServer.Core.PageData:

public static T ShallowCopy<T>(PageData copy)
where T: PageData, new();


Objections anyone?

We are now interested to hear if there are anyone that strongly disagrees with the suggested change.

Is there anyone out there that uses the APIs listed above?

If yes, would it be sufficient for you to replace the DataFactory changes with equivalent extension methods with the same signature and same semantics?

Please, post a comment here or drop me a mail (fredrik [dot] tjarnberg [at] episerver [dot] com).

Jan 25, 2010


Björn Olsson
Björn Olsson Sep 21, 2010 10:33 AM

No objections your honour. Haven't used them myself either. I would prefer some new, optimized and cached methods/ways to retrieve pages filtered by categories and so on. Instead of having to implement this logic on top of EPiServers API. I'm not a huge fan of FindPagesWithCritera, or similar aproaches (spelling?) :)

Sep 21, 2010 10:33 AM

I've used PageTypeBuilder on more-or-less every project since it's release in the summer last year and love working with EPiServer in this way and certainly wouldn't go back.

I personally have got round the issue described by Joel by creating extension methods on the PageData and PageReference objects instead (without going into too much detail). However, I'm fully behind Joel's request to have these superfluous methods removed - they can easily be replaced as you say with some custom extension method code anyhow. It may however come back to bite someone though when they upgrade to 6 - but I guess the release notes will have to cover this :-)

Please login to comment.
Latest blogs
How to have a link plugin with extra link id attribute in TinyMce

Introduce Optimizely CMS Editing is using TinyMce for editing rich-text content. We need to use this control a lot in CMS site for kind of WYSWYG...

Binh Nguyen Thi | Jul 13, 2024

Create your first demo site with Optimizely SaaS/Visual Builder

Hello everyone, We are very excited about the launch of our SaaS CMS and the new Visual Builder that comes with it. Since it is the first time you'...

Patrick Lam | Jul 11, 2024

Integrate a CMP workflow step with CMS

As you might know Optimizely has an integration where you can create and edit pages in the CMS directly from the CMP. One of the benefits of this i...

Marcus Hoffmann | Jul 10, 2024

GetNextSegment with empty Remaining causing fuzzes

Optimizely CMS offers you to create partial routers. This concept allows you display content differently depending on the routed content in the URL...

David Drouin-Prince | Jul 8, 2024 | Syndicated blog