<img alt="Liferay" height="36" src="http://cdn.www.liferay.com/osb-www-theme/images/custom/heading.png" width="146" />

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Hi,
Anyone know if there is a plan to upgrade to latest JAMWiki, 1.2.2 released last August?
http://jamwiki.org/wiki/en/JAMWiki_Release_Archive

I saw an Improvement-type issue was created for upgrading to 0.9.3, but none is there for 1.2.2; any plans to upgrade?
http://issues.liferay.com/secure/IssueNavigator.jspa?reset=true&jqlQuery=%28summary+~+jamwiki+OR+description+~+jamwiki+OR+comment+~+jamwiki%29+AND+issuetype+%3D+Improvement

Thanks.
Bijan

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 8.12

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Hi Bijan,

I don't think there are short term plans to do this. In fact, I think the support to JAMWiki should be taken out of the core and moved to a plugin.

Would you be able to help with that task and also upgrade to the new version?

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 8.40

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Hi Jorge,

Yes, I would be happy to help with that task.

Thanks.
Bijan

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 8.47

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Awesome. Could you give it a first try at extracting it out and send me a pull request?

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 8.55

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Good idea; I'll decouple Wiki plugin from core and into it's own plugin.

I expect I can have something ready next Sunday 2013/01/20.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 9.22

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Great.

BTW, note that I'm only talking about decoupling the support for JAMWiki, not the whole wiki portlet.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 9.36

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Oh, ok. I see what you mean - create new JAMWiki portlet reusing code from Liferay core, correct?

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 10.02

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Not exactly. JAMWIki is used in the implementation of one of the formats implemented for Liferay's Wiki: MediaWiki. So my proposal is to make a plugin of type hook that adds support for that format (currently it's included in the core and is enabled through portal.properties instead).

Makes sense?

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 10.20

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

To verify, extract mediawiki portal properties, e.g. below, and enable them via a hook, e.g media-wiki-hook?

#
# Set the list of supported wiki importers.
#
wiki.importers=MediaWiki

#
# Configure settings for each of the wiki importers.
#
wiki.importers.page=/html/portlet/wiki/import/mediawiki.jsp
wiki.importers.class=com.liferay.portlet.wiki.importers.mediawiki.MediaWikiImporter

Correct me if I'm wrong, the last property is not compatible to be in a hook since the MediaWikiImporter.class file would be in the hook's classloader not the portal's. As such, the mediawiki.jsp references would fail, right? Also, the com.liferay.portlet.wiki.translators.MediaWikiToCreoleTranslator depends on the MediaWikiImporter class.

Thanks for explaining this proposal.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 10.53

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Hi Bijan,

Yes, this is exactly what I had in mind. I understand that you may need to do some changes to the implementation and maybe add some additional "hookability" that is not already there. But there should be some way of doing it that will also be helpful for others who may want to add other formats. So your work will be doubly useful emoticon

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 gennaio 2013 11.03

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Hi Jorge,
Ok, thanks for the verification and guidance.
I expect to have something by 2013/01/20 still (lots on plate this weekend.)
Thanks again.
Bijan

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

21 gennaio 2013 11.44

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Hi Jorge,
I succesfully extracted out the Jamwiki-related code/binaries into a hook, but ran out of time while adding new hookability feature for adding wiki engines. Would you mind commenting if following way is ok?
Basically, add some logic to the com.liferay.portal.deploy.hot.HookHotDeployListener.initPortalProperties method. Namely, to add something like:

1
2//pseudocode
3if ( isWikiProperties ) {
4 // load the wiki classes referenced
5 // update wiki engines / formatters / importers
6 // update jsps (e.g. to remember selection of this new format
7}

Also, opened LPS-32447 to track progress
My commits will be available in about 30 minutes (after they're uploaded on GitHub) at following URIs:
Liferay Portal changes
Liferay Plugins changes

Thanks.
Bijan

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

22 gennaio 2013 0.49

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Hey Bijan,

I've been taking a look at your code and it looks very good so far.

Regarding your question, we shouldn't add any code to HookHotDeployListener that is specific to these properties. What you are mentioning about the classloading is correct, but it's the same for all hook properties that also reference classes. I haven't developed a hook in a while so I cannot tell you exactly how it works, but you can take a look at the other hooks in the plugins repository for examples.

Let me know if you can't find how to do it and I'll try to look it up.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

22 gennaio 2013 2.00

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Ignore my last post, your approach is the only one possible at this point. We are working on an OSGi Liferay Module framework that will allow us to avoid this type of manual classloading handling, but for now you will have to modify HookHotDeployListener to add code to load the importer and formatter classes. You should add it around line 1957 (follow alphabetic order with respect to the property being read). Use the examples above for reference, for example:

1       if (portalProperties.containsKey(
2             PropsKeys.USERS_SCREEN_NAME_VALIDATOR)) {
3
4          String screenNameValidatorClassName = portalProperties.getProperty(
5             PropsKeys.USERS_SCREEN_NAME_VALIDATOR);
6
7          ScreenNameValidator screenNameValidator =
8             (ScreenNameValidator)newInstance(
9                   portletClassLoader, ScreenNameValidator.class,
10                   screenNameValidatorClassName);
11
12          ScreenNameValidatorFactory.setInstance(screenNameValidator);
13       }

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

22 gennaio 2013 11.23

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Really good point about the OSGi Liferay Module; I will go ahead with the manual classloading approach and have an update latest by EOD Sunday.
Thanks Jorge.

Jorge Ferrer:

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

31 gennaio 2013 20.04

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Hi Jorge,
So sorry - I have not had a chance to work on this, but will definitely cut out the time to do so this weekend.
Thanks for your patience.
Bijan

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

1 febbraio 2013 4.39

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

No worries, looking forward to see it.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

9 febbraio 2013 9.47

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Thanks again for your patience.
Just wanted to let you know I have not forgotten and will cut out time to do work on it today and tomorrow.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

17 febbraio 2013 23.09

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

FYI, I'm documenting the steps in the wiki entry at the following URI:
http://www.liferay.com/community/wiki/-/wiki/Main/jamwiki
and committing changes to:
Liferay Portal - https://github.com/bmvakili/liferay-portal/tree/lps-jamwiki
Liferay Jamwiki Hook - https://github.com/bmvakili/liferay-plugins/tree/lps-jamwiki
Currently the Portal is building and I'm debugging to make sure the Hook builds and deploys well.
Finally, I will do some QA/testing to make sure this doesn't break something.
Thanks for your patience.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

19 febbraio 2013 2.17

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Awesome, Thanks Bijan. Looking forward for your pull request emoticon

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

2 marzo 2013 23.18

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

Updated code; jamwiki-hook deploys (and works!) fine, but I'm still working on the undeploy/destroy code; I need to clean it up a little more before it's ready for review, but I expect some time this week to finally hand this off to you.
Thanks for your patience.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

4 marzo 2013 6.48

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

No worries. Thanks for your work Bijan!

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

3 aprile 2013 5.01

Risposta

Punteggio: Regular Member

Messaggi: 120

Data di Iscrizione: 10 marzo 2009

I am very close to finishing it;
Was delayed since my two week vacation.
All that is left are the destroy methods:
Liferay Portal - https://github.com/bmvakili/liferay-portal/tree/lps-jamwiki
Liferay Jamwiki Hook - https://github.com/bmvakili/liferay-plugins/tree/lps-jamwiki
Thanks.

Contrassegno come una risposta

RE: Upgrading to JAMWiki 1.2.2?

4 aprile 2013 0.55

Risposta

LIFERAY STAFF

Punteggio: Liferay Legend

Messaggi: 2725

Data di Iscrizione: 31 agosto 2006

Looking forward to it. Feel free to send me pull requests whenever you want me to take a look at it.