Adding function "createNew" to simplify the creation of a ezsitedata entry

[pkamps]

Most changes are due to code reformatting. I added a new function "createNew" to simplify the creation process.
The function has a new unit test.

[peterkeung]

• Should be using spaces instead of hard tabs
• This should be submitted to the main eZ repo as well
• Should call the function "create" to match other persistent objects? Usually, though, only the row gets created and then you have to call "store()" outside of the class: http://share.ez.no/learn/ez-publish/a-quick-and-friendly-introduction-to-ezpersistentobject/(page)/3. That is, in fact, what your unit test suggests (so right now you are calling "store()" twice)

[pkamps]

Ha, it's so good to have people reviewing your code.

1. yes, just yes
2. No, it's wasting my time - I don't mind if somebody else submits it.
3. I looked at other persistentobject classes like ezpolicy.php or ezrole.php: they also call 'store()' inside the class in case of the function 'createNew'. But a lot more kernel classes implement the 'create' function. So I think I'll drop 'createNew' and implement a 'create' function. Or do you think we should keep the createNew function?

[peterkeung]

2. We will discuss more internally. There is a big picture here, which is that the eZ subscription clients are using eZ's repo.
3. I like a "create" function without "store()". In keeping with the convention with normal eZ content objects, we then add "store()" in a separate API layer. It's a bit silly in my opinion, but it's at least consistent with what we use most often.

[pkamps]

Point 1) and 3) addressed.

[ernestob]

I have tested this, it works great and it is very flexible. This seems to be a good replacement for objects created in the CMS with a focus on storing data that is not going to be directly edited by the users.

Creating a value and creating a new value with the same key(name) will overwrite any existing value.

[peterkeung]

+1 other than these doc fixes

[peterkeung]

"Allows" should be "Enables you"

[peterkeung]

"Constructs" not "Cronstructs"

[pkamps]

Typos fixed. Please confirm.

[peterkeung]

Good to squash and merge.

[pkamps]

ezp pull request:
ezsystems#1273