Forum OpenACS Development: Re: creating a new xowiki class

4: Re: creating a new xowiki class (response to 3)

Posted by Gustaf Neumann on 08/19/08 12:42 PM

This looks to me to complicated.

The question is how to characterize these "few cases". I assume for now, that the "few cases" can be characterized as a type of application, therefore subclassing the xowiki app (like in s5) looks appropriate. i would do the following way:

1) subclass xowiki::Package as ::myapp::Package
2) in mayapp-procs.tcl, define in addition to ::myapp::Package the class ::myapp::MyWikiForm with the intended methods (use "next" in the methods to chain to the default behaviour if necessary/intended)
3) in the method initialize of ::myapp::Package, make a "::xowiki::MyForm instmixin add ::myapp::MyWikiForm; next"
4) in the method destroy of ::myapp::Package, do a "::xowiki::MyForm instmixin delete ::myapp::MyWikiForm; next"

There is no duplication of code involved.

5: Re: creating a new xowiki class (response to 4)

Posted by Alvaro Rodriguez on 08/19/08 03:27 PM

This time things started to sound more clear to me, I'm still getting to know xotcl and all that stuff, Today I'm gonna try to implement what you suggested and I'll let you know.

Thank you very much Gustaf

6: Re: creating a new xowiki class (response to 4)

Posted by Alvaro Rodriguez on 08/20/08 12:50 AM

I did what you suggested and it worked, I managed to mixin the Forms and add the wanted behavior, but I had a few complications:

- If I restart the server, it loads my procs file first and throws an error because it needs some other definitions on the next files (not a big deal).
- To test my new app right now I changed the index.vuh so instead of call the initialize from ::xowiki::Package it calls ::myapp::Package but I need to leave it independent from the rest of the pages, I tried using another .vuh file inside another folder or using a .tcl file but it always says that "page" is not available. (the xowiki message if I want to create that page)

Any suggestions,

Thank you very much for your help Gustaf,

7: Re: creating a new xowiki class (response to 6)

Posted by Stefan Sobernig on 08/20/08 11:31 AM

Alvaro!

- If I restart the server, it loads my procs file first and throws an error because it needs some other definitions on the next files (not a big deal).

The OpenACS package initialisation procedure is rather hostile to the kind of package refinement you are probing; due to its alphanumerical sorting of packages/files to be evaluated upon start-up.

Place the following in the first *-procs.tcl in your xowiki-derived package:

::xo::db::require package xowiki

this will source xowiki before your package, so everything needed is in place ...

- To test my new app right now I changed the index.vuh so instead of call the initialize from ::xowiki::Package it calls ::myapp::Package but I need to leave it independent from the rest of the pages, I tried using another .vuh file inside another folder or using a .tcl file but it always says that "page" is not available. (the xowiki message if I want to create that page)

this one i don't quite understand (that is, what you try to achieve). would you mind posting your sources somewhere, preferably the entire package (public scm, archive), so we can have a look. it's a thousand times easier to comment on code than to comment on comments on invisible code ...

//stefan

8: Re: creating a new xowiki class (response to 6)

Posted by Gustaf Neumann on 08/20/08 02:19 PM

Alvaro,

Maybe the prototype pages for your package is missing, so you don't get a useful index page. I have added a small section to the xowiki tutorial http://alice.wu-wien.ac.at:8000/s5-xowiki-tutorial/slides#subclassing-xowiki

Hope this helps
-gustaf neumann

9: Re: creating a new xowiki class (response to 7)

Posted by Alvaro Rodriguez on 08/20/08 03:36 PM

Thanks for the help Stefan,

I know the second part sounds confusing but is actually not that confusing.

The xowiki's index.vuh does this:

::xowiki::Package initialize -ad_doc {
} ...

and I changed it to do this:

::myapp::Package initialize -ad_doc {
} ...

So now when I call a xowiki/?edit-new=1 it actually initializes myapp and it works, but I can't leave the index.vuh file like this.

My question is how could I set a file to work the way I want, should I use a .vuh. I tried using a tcl file with that code but when it founds the part of the code:

::myapp::Package initialize -ad_doc {
} ...

it ignores the file.

10: Re: creating a new xowiki class (response to 8)

Posted by Alvaro Rodriguez on 08/20/08 03:40 PM

Thank you Gustaf,

what you added in that section is exactly what I did and it worked for me.

11: Re: creating a new xowiki class (response to 9)

Posted by Alvaro Rodriguez on 08/20/08 07:19 PM

I checked the s5 package and noticed that I wasn't doing things right, my procs file was on the /xowiki directory and it should be in /myapp directory along with my new index.vuh

12: Re: creating a new xowiki class (response to 11)

Posted by Gustaf Neumann on 08/20/08 08:10 PM

right... as described in the tutorial section... :)

everything ok by now?

13: Re: creating a new xowiki class (response to 12)

Posted by Alvaro Rodriguez on 08/20/08 08:20 PM

I read the tutorial but the first time I missed that.

I haven't tried that yet, I need to do a little extra work first but I'll check the s5 package to guide my work. If I need any help I'll let you know...

Thanks for all the help,

14: Re: creating a new xowiki class (response to 7)

Posted by Alvaro Rodriguez on 08/21/08 07:54 PM

Everything worked out well, but for some reason I keep getting the same error about the loading packages.

I placed the line "::xo::db::require package xowiki" at the beginning of myapp-procs.tcl (the only procs file), I did it exactly the same way as the s5 package but it keeps failing.

Is there anything else I need to check for this ...