24 Jul
2007
24 Jul
'07
9:03 p.m.
Hi Sergiu,
On 7/25/07, Sergiu Dumitriu <sergiu.dumitriu(a)gmail.com> wrote:
On 7/24/07, Asiri Rathnayake <asiri.rathnayake(a)gmail.com> wrote:
Agreed.
Still, if there's an error while deleting documents, the server method
Hi All,
We were silent for few days due to mid-semester work load (still have
some
unfinished business). Anyway, I have added the
removeSpace RPC to xwiki
back-end. But now i'm having few doubts about it. The remove space RPC
works
by retrieving all the page names under the given
space and deleting them
individually. But following complications may happen when using this
RPC,
1. If the user does not have enough privileges to delete some of the
pages
within that space, an exception will be thrown
half-way down. Now the
RPC
invoker does not know which pages were removed
and which were not.
(Thus, he
is out of sync with the wiki)
2. If instead we chose to remove page by page from the client - Xeclipse
(say), we don't have this problem , and we could even display a progress
bar. (to indicate that pages are being deleted).
Since the XML RPC API requires the removeSpace() RPC, we must indeed add
it.
But wouldn't it be better if we do not use
this RPC within Xeclipse (due
to
reasons mentioned above) ?
Or, is there a better way to implement removeSpace() RPC ? (like
checking
privileges before doing anything ?) ) (+ How ? )
From the user PoV, it is better to have a feedback as documents are
deleted. This means that the client should make requests one by one.
From a developer PoV, this means a lot more traffic and a lot more
time, as each request creates a new connection, a new context
initialization, new objects, etc. should not be aborted, but should keep a list of
failed documents.
Now, I don't know what to do with this list. Should the method return
a List, and if it is empty then everything was OK? Or should we simply
return true/false, and make another call to see what documents are
left in the space, if the result was "false"?
These are the types of complications that i came across. Although we can
come up with some workaround like making a second call to sync the client
with server, we can avoid all that trouble by simply deleting page by page.
But
again that is little bit (?) more over-head than a single RPC.
I prefer deleting page by page since that would make our life easy by not
having
to worry about complex state management on the client (like making a second
RPC
and refreshing the whole object model on the client so that it corresponds
to server).
Thanks.
- Asiri
Sergiu
--
http://purl.org/net/sergiu
--
You receive this message as a subscriber of the xwiki-dev(a)objectweb.orgmailing list.
To unsubscribe: mailto:xwiki-dev-unsubscribe@objectweb.org
For general help: mailto:sympa@objectweb.org?subject=help
ObjectWeb mailing lists service home page: http://www.objectweb.org/wws