On Sep 8, 2008, at 8:40 PM, Sergiu Dumitriu wrote: > Vincent Massol wrote: >> Hi Sergiu, >> >> On Sep 5, 2008, at 1:17 AM, Sergiu Dumitriu wrote: >> -0 (close to -1) till you explain what is wrong with the solution >> below :) >> >> This could also be implemented using a special Tag (like: >> "xwiki:hidden" for example). > > The problem is that we want to leave out the documents that are NOT > hidden, so a query that does that will have to use a subquery (either > NOT EXISTS or NOT IN), which increases the query complexity and > execution time. And, as Jerome said, we'd have to filter this tag when > displaying documents, and, as you said, this setting should not have > the > same rights as normal tags. > >> I was going to say: "In general I think we need the ability to add >> generic metadata to a document.". however this already exists: it's >> called Objects! :) >> So I think instead of adding more and more fixed metadata we might >> instead want to use Objects. Either reuse the existing Tag or >> create a >> new Visibility class/objects. >> >> What are the cons of using Objects for this feature? > > Mostly query complexity and execution time, but also the fact that for > the moment this should be a "private" setting that should not be > accessible to normal users. As I said, this is a hack until we'll have > the new model in place. The hack is needed because I don't want to use > the 'XWiki' space for all support documents (macros, templates, style > and others).
ok, +0 for now knowing it's a hack that we need to fix later on. Would be good to document it in the javadoc as such. Thanks -Vincent > The extra document metadata field is not such a bad idea > IMHO because it is the least intrusive solution. Yes, I know it is the > most intrusive one (on the implementation level) because it changes > lots > of things, but from the users' PoV it is invisible. > -- > Sergiu Dumitriu > http://purl.org/net/sergiu/ _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
