Hi Caleb & Sergiu,
/**
+ * Loads an Document from the database. Rights
are checked on the
author (contentAuthor) of the document
+ * containing the currently executing script before sending back the
loaded document.
+ *
+ * @param fullName the full name of the XWiki document to be loaded
+ * @return a Document object or null if it is not accessible
+ * @throws XWikiException
+ * @since 2.3M1
+ */
+ public Document getDocumentAsAuthor(String fullName) throws
XWikiException
+ {
+ DocumentReference reference;
+
+ // We ignore the passed full name if it's null to match behavior
of getDocument
+ if (fullName != null) {
+ // Note: We use the CurrentMixed Resolver since we want to use
the default page name if the page isn't
+ // specified in the passed string, rather than use the current
document's page name.
+ reference =
this.currentMixedDocumentReferenceResolver.resolve(fullName);
+ } else {
+ reference = this.defaultDocumentReferenceResolver.resolve("");
+ }
+
+ return getDocument(reference);
This doesn't make sense, silly copy paste probably, shouldn't it be return
getDocumentAsAuthor(reference); ?
> + }
On Tue, Mar 30, 2010 at 04:28, Sergiu Dumitriu <sergiu(a)xwiki.com> wrote:
> I wonder if this is the best way to get the
content author. What happens
> with all the different include methods? Specifically, DocumentA includes
> DocumentB (sheet or topic), the call is in DocumentB; what is the
> context document?
> > + String author =
getXWikiContext().getDoc().getContentAuthor();
Please have a look at XWIKI-5027, it is the right place for discussing that
issue.
Caleb changes does not make anything worse, but this is not perfect for
sure.
Denis
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO