Just saw that for example the calendar app is using `OC.Share.share()`.
I think the safest is to keep a single function but with two argument lists.

1) The old argument list: itemType, itemSource, shareType, shareWith, permissions, itemSourceName, callback
2) The new arguments: options (hash array of argument name to values)

To make it work for both, you can check "if (arguments.length > 2)" you could assume that it's 1). "if (arguments.length === 1)" then it's 2) and convert the values accordingly.

Let me know if you need more info.