Hi Ruslan,

On Jan 11, 2008, at 3:55 PM, rssh wrote:

>
> Good day, xwiki community.
>
> I want to discuss path, with allows to set in configuration name of  
> main wiki
> database schema and prefixes for databases schemes of virtual wiki-s.
>
> This allows to use few wiki-based applications on one computer.
>
> Proposed path itself is attached.
> The work of this path is
> 1. add configuration variables:
> xwiki.db  --  database schema name of main wiki instance.
> xwiki.virtual.db.prefix  -- prefix for  database scheme-s of virtual  
> wiki-s.
>
> So, questions:
> a)  where I must write documentation for this parameters ?
> b)  can I submit JIRA issue now or exists some other things which i  
> forgott.

Thanks for the patch. I've looked at it quickly and here are some  
comments:

* Could you please ensure you're following the xwiki coding  
conventions as defined here:
http://dev.xwiki.org/xwiki/bin/view/Community/CodeStyle. You can also  
read this page: http://dev.xwiki.org/xwiki/bin/view/Community/Contributing

* Make sure you don't include diffs for stuff not related to your  
changes. For example in your patch you have modified a license header,  
you've also modified lots of other places (I think you changed the  
line returns).

* We already have a method for returning configuration parameters.  
It's called XWiki.Param() (Note: with a wrong uppercase letter for a  
method! We need to fix this).

Have you tested it in virtual mode (multiwiki)? This is an area that  
should generate some problems I believe.

Now to answer your questions:

a) You can document it in the code itself and on xwiki.org here:
http://platform.xwiki.org/xwiki/bin/view/AdminGuide/Configuration

b) yes for jira definitely.

Thanks!
-Vincent




_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to