1 Dec
2009
1 Dec
'09
2:38 p.m.
On Tue, Dec 1, 2009 at 13:55, Asiri Rathnayake
<asiri.rathnayake(a)gmail.com> wrote:
Hi Devs,
in XWiki::initXWiki(XWikiConfig config, XWikiContext context,
XWikiEngineContext engine_context, boolean noupdate) method signature, the
flag "noupdate" indicates whether the database should be updated or not. But
as far as I can understand, it's meaning is reversed:
- if "noupdate" is true -> You can update the database and install data
into
it (like define major xwiki classes etc.).
- if "noupdate" is false -> You should not add anything into the database.
(I guess this means you can only create the database schemas.)
One observation I made though is that "noupdate" is actually used only in
the following code segment:
<code>
// Make sure these classes exists
if (noupdate) {
initializeMandatoryClasses(context);
getStatsService(context);
}
</code>
* Question:- Do we really need this flag?
The other problem is that pckager plugin is using a version of XWiki
constructor that sets this flag to "false" by default:
see
http://svn.xwiki.org/svnroot/xwiki/platform/xwiki-tools/trunk/xwiki-package…
http://svn.xwiki.org/svnroot/xwiki/platform/core/trunk/xwiki-core/src/main/…
proof.
This means when we are packaging the database (hsqldb), none of the
mandatory classes are packaged. They will only be initialized when XE is
loaded for the first time.
* Question:- Is this the desired behavior?
This mean we have objects but not the corresponding classes in the
database so the database is not valid. Even if XE is able to fix the
issue it seems wrong to be to generate an invalid database.
Apperantly I have introduced a bug (sort of) because XWiki.WikiMacroClass &
XWiki.WikiMacroParameterClass are initialized inside if(noupdate){} clause
but I have invoked registerWikiMacros(); call outside of it (in
XWiki::initXWiki()). This makes hsqldb database build report an exception
because wiki macro registration code throws an exception if it cannot find
the necessary xwiki class definitions.
I can fix this error by simply moving registerWikiMacros(); call inside the
if(noupdate){} check but I'm not sure whether it is the correct way.
WDYT?
IMO we should enable standard class generation in packager.
Now for the registerWikiMacros(); issue i see to issues actually:
* it does not sounds right to have wiki macro classed initialized in
initializeMandatoryClasses in fact (I even tough it was done in
registerWikiMacros()), for me it's not a mandatory class, it should be
initialized by the wiki macro initializer
* since "noupdate" seems related to "don't put data in the database,
just structure" yes registerWikiMacros() should go inside the if block
- Asiri
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
--
Thomas Mortagne