IRC Archive for channel #xwiki on 25 August 2015

Last modified by Vincent Massol on 2015/08/25 23:22

<abusenius_> has quit
00:45 <Pbas> has quit
00:50 <martin-h> has quit
00:52 <martin-h> has joined #xwiki
00:53 <vmassol> has joined #xwiki
00:58 <Pbas> has joined #xwiki
00:59 <vmassol> has quit
01:11 <Chuguniy> has joined #xwiki
04:35 <cpe> has quit
04:36 <cpe_> has joined #xwiki
04:36 <cpe_> is now known as <cpe>
05:08 <Chuguniy> has quit
05:14 <martin-h> has quit
05:15 <martin-h> has joined #xwiki
05:25 <_cjd_> has quit
05:26 <_cjd_> has joined #xwiki
05:43 <mflorea> has joined #xwiki
07:11 <Ramona1> has joined #xwiki
07:22 <martin-h> has quit
07:22 <Ramona1> has quit
07:24 <martin-h> has joined #xwiki
07:54 <_init_cjd> has joined #xwiki
07:57 <_cjd_> has quit
08:05 <gsmeria> has joined #xwiki
08:10 <_init_cjd> has quit
08:15 <Ramona2> has joined #xwiki
08:19 <sorinello> has joined #xwiki
08:19 <sorinello> has joined #xwiki
08:30 <msmeria> has joined #xwiki
08:39 <evalica> has joined #xwiki
08:42 <tmortagne> has joined #xwiki
08:43 <vmassol> has joined #xwiki
08:56 <Ramona2> has quit
08:56 <Ramona2> has joined #xwiki
09:01 <_init_cjd> has joined #xwiki
09:13 <KermitTheFragger> has joined #xwiki
09:59 <Ramona2> has quit
09:59 <Ramona3> has joined #xwiki
10:02 <gdelhumeau> has joined #xwiki
10:49 <Trefex> has joined #xwiki
11:00 <tmortagne> has quit
11:22 <gsmeria> has quit
11:22 <Ramona3> has quit
11:23 <Ramona2> has joined #xwiki
11:37 <_init_cjd> I just found an issue w/ 7.2 zipexplorer
11:38 <cjd> has joined #xwiki
11:42 <_init_cjd> has quit
12:18 <vmassol> FYI, starting to work on the oracle bug for 7.2M3
12:18 <vmassol> http://jira.xwiki.org/browse/XWIKI-12425
12:19 <cjd> Is there a known issue with zipexplorer on 7.2 ?
12:19 <vmassol> i don't tink so cjd
12:19 <vmassol> *think
12:20 <cjd> ok heads up that I notices on 7.2 I'm getting this:   http://81.220.71.54:8080/xwiki/bin/download/SyntaxHighlighting/SyntaxHighlighting/codemirror.zip/codemirror-3.14/lib/codemirror.js --> 404
12:20 <vmassol> this plugin has not been touched in the past 8 years or so :)
12:20 <cjd> and the zip exists and it has the file in it
12:20 <vmassol> ah
12:20 <vmassol> wait
12:20 <vmassol> thomas has updated commons compress
12:20 <vmassol> maybe we use that
12:20 <vmassol> in this plugin
12:20 <cjd> was thinking more like path parsing or something...
12:21 <vmassol> hmm no
12:21 <vmassol> ah yes you're right
12:21 <vmassol> It must be me
12:21 <vmassol> who broke it when intoducing url schemes
12:21 <vmassol> a jira issue would be nice
12:22 <cjd> can do
12:22 <vmassol> the problem must exist since 6.x
12:22 <cjd> will test w/ 7.1
12:22 <vmassol> try with 6.4
12:22 <vmassol> it should fail too IMO
12:22 <vmassol> or 6.4.4
12:23 <cjd> also battling an odd issue where the wrong version of Syntax Highlighting Application is being installed
12:23 <cjd> but Yann does not have the issue (he's on Windows, I'm on Mac, everything else is equal)
12:23 <cjd> right down to the .zip file we unpack
12:25 <cjd> so Extension Manager is installing Syntax Highlighting 2.2 for me and 3.1 for him (it should be detecting the XWiki version and installing 3.1)
12:26 <cjd> Testing now on Linux to see if it's a Mac issue...
12:27 <cjd> same issue on linux
12:28 <cjd> http://openpass-recoengin.devxwiki.com:8080/xwiki/bin/download/SyntaxHighlighting/SyntaxHighlighting/codemirror.zip/codemirror-3.14/lib/codemirror.js
12:28 <cjd> 7.1.2 -> works
12:29 <cjd> http://81.220.71.54:8080/xwiki/bin/download/SyntaxHighlighting/SyntaxHighlighting/codemirror.zip/codemirror-3.14/lib/codemirror.js
12:29 <cjd> (You'll have to trust me on that one)
12:29 <cjd> 7.2-milestone-2 -> fails
12:29 <vmassol> ok
12:30 <cjd> creating issue
12:33 <cjd> http://jira.xwiki.org/browse/XWIKI-12438
12:33 <vmassol> thanks
12:34 <cjd> sure thing
12:34 <cjd> still tracking down this issue of versions being wrong
12:39 <vmassol> lunch bbl
12:41 <gsmeria> has joined #xwiki
12:52 <Enygma`> has joined #xwiki
13:53 <Ramona3> has joined #xwiki
13:53 <Ramona2> has quit
13:58 <vmassol> has quit
13:59 <vmassol1> has joined #xwiki
14:02 <ClemensR> has joined #xwiki
14:08 <vmassol1> ok I've reproduced XWIKI-12425 now debugging
14:20 <Ramona3> has quit
14:20 <Ramona2> has joined #xwiki
14:41 <vmassol1> mflorea: FTR lucene and solr 5.3.0 are out :)
14:42 <mflorea> ok
14:42 <vmassol1> is now known as <vmassol>
14:43 <vmassol> we should probably wait for 7.3 to upgrade…
14:50 <martin-h> has quit
14:53 <martin-h> has joined #xwiki
14:54 <mflorea> yes
15:22 <gsmeria> has quit
15:23 <gsmeria> has joined #xwiki
15:28 <Ramona2> has quit
15:29 <Ramona2> has joined #xwiki
15:29 <Ramona2> has quit
15:30 <Ramona2> has joined #xwiki
15:46 <tmortagne> has joined #xwiki
15:55 <gsmeria> has quit
15:56 <gsmeria> has joined #xwiki
15:56 <gsmeria> has left #xwiki
15:59 <gsmeria> has joined #xwiki
16:07 <Trefex> hey tmortagne vmassol ClemensR. it's me :)
16:07 <Trefex> I created my second sub-wiki, and the process is stuck at Installing document [https://lcsb-wiki.uni.lu/xwiki/wiki/iman/view/Main/ » https://lcsb-wiki.uni.lu/xwiki/wiki/iman/view/Stats/ » https://lcsb-wiki.uni.lu/xwiki/wiki/iman/view/Stats/Style]
16:07 <Trefex> however on another tab, i can login, is there some heavy duty background task going on?
16:11 <vmassol> still 25 issues left for 7.2M3 :(
16:11 <vmassol> and still 7 blockers
16:12 <Trefex> it's stuck on Installing document Stats >> Style and htop reports 200% CPU usage on jave processes, something is weird
16:12 <martin-h> has quit
16:13 <Trefex> do you have any ideas what I could try to do?
16:13 <vmassol> moved 2 blockers for RC1
16:14 <vmassol> Trefex: sorry I'm not an expert in DW
16:14 <vmassol> (or EM)
16:14 <Trefex> what's DW or EM vmassol ?
16:14 <martin-h> has joined #xwiki
16:15 <vmassol> distribution wizard
16:15 <vmassol> extension manager
16:15 <Trefex> ah, is this also used when you create a new sub-wiki ?
16:15 <Trefex> i guess so
16:24 <vmassol> mflorea: you're our JS expert and I don't feel confident implementing XWIKI-12205 alone by myself, I'd need at least some guidance from you and probably more than that. ATM I'm starting working on XWIKI-12438 since I know you're busy but we need to fix 12205 for M3 since it causes several other provlems
16:24 <vmassol> (TBH i'd be more confident that someone more knowledgable about xwiki js apis work on it than myself)
16:25 <vmassol> (but I'm ok to learn and try if need be, with someone who know js to guide me ;))
16:28 <vrachieru> has joined #xwiki
16:29 <msmeria> has quit
16:37 <OSIMasson> has joined #xwiki
16:44 <Trefex> vmassol: the snippet http://extensions.xwiki.org/xwiki/bin/view/Extension/Gravatar+Import
16:44 <Trefex> doesn't work in xWiki 7.1.2, as is, i get a Nullpointer
16:45 <vmassol> interesting
16:45 <vmassol> it should work unless we've broken some apis
16:45 <vmassol> I'll check it
16:46 <vmassol> works for me
16:46 <vmassol> except for one small detail
16:47 <vmassol> I've used services.query.xwql() instyead of the deprecated api in the snippet
16:47 <vmassol> (for which you need a legacy build)
16:47 <vmassol> snippet updated
16:52 <Enygma`> has quit
16:52 <Trefex> vmassol: thanks
16:52 <vmassol> works for you?
16:52 <Enygma`> has joined #xwiki
16:53 <Trefex> vmassol: i'll try now
16:53 <Trefex> nope, same
16:54 <Trefex> https://paste.fedoraproject.org/259039/44051443/
16:55 <vmassol> I think it just means you have some users without an "email" proerty
16:56 <vmassol> seems the null is at  def email = userObj.getProperty("email").getValue()
16:56 <vmassol> when calling getValue() on some null
16:56 <vmassol> so getProperty must have returned null
16:56 <vmassol> so email is probably not existing
16:56 <Trefex> hah
16:57 <Trefex> <— idiot
16:57 <Trefex> i didn't do the email LDAP mapping
16:57 <Trefex> ...
16:58 <Trefex> vmassol: do you have somewhere the full list of user properties?
16:58 <vmassol> in the XWiki.XWikiUsers xclass
17:01 <Trefex> not sure how to find those
17:02 <vmassol> go to that page and edit using the class editor
17:04 <Trefex> gotcha
17:05 <vrachieru> has quit
17:06 <mflorea> vmassol: I'd like to help but I'm busy with integrating the tree in the create/copy/rename UI. I should be able to finish today, but there is still some work needed to make the copy/rename async (with progress), similar to what gdelhumeau did for the delete action
17:07 <gdelhumeau> mflorea: I am thinking about adding the logs in the REST JobStatus
17:07 <mflorea> would be good
17:07 <gdelhumeau> but I'm affraid that the JSON would become too loud
17:08 <vmassol> verbose?
17:08 <gsmeria> has left #xwiki
17:08 <Slashman> has joined #xwiki
17:10 <mflorea> vmassol: so either gdelhumeau/Enygma` help with you XWIKI-12205, or they help me with the progress bar on the copy/rename and I help you (tomorrow, after I commit my work), or we add the progress bar in RC1 and I help you tomorrow
17:10 <vmassol> :)
17:10 <vmassol> tomorrow is fine
17:10 <Trefex> vmassol: it's not possible to sync all ldap accounts or? eg i know added email mapping
17:10 <Trefex> but it only updated my own account, not for all ldap entries
17:10 <mflorea> ok :)
17:10 <vmassol> I need some time to finish XWIKI-12438
17:10 <vmassol> it's complex actually
17:11 <gdelhumeau> I'm doing a test with a delete of 500 documents, and the log is ~132 ko in JSON
17:11 <gdelhumeau> the full log
17:11 <vmassol> Trefex: no idea, I don't use LDAP
17:12 <cjd> zipexplorer and nested spaces just don't get along that well
17:12 <Trefex> anybody else can help me with LDAP?
17:12 <vmassol> cjd: "The real fix is that the zipexplorer plugin would need its own action instead of reusing the download one..."
17:12 <gdelhumeau> but we don't often delete 500 documents so maybe it's ok to have a big log
17:12 <vmassol> what doesn't get along is the old plugin mechanism and the new URL schemes
17:12 <tmortagne> Trefex: synchronization happen only on the user that login
17:12 <cjd> yeap
17:13 <vmassol> old plugin interface had a downloadAttachment() method
17:13 <vmassol> accepting an attachment
17:13 <Trefex> tmortagne: crap, so now i can't do gravatar snippet, because each user have to login first
17:13 <vmassol> the reason it workes before was because of a hack in the download action
17:13 <vmassol> *worked
17:14 <vmassol> that action was supporting/ignoring variable-lenght paths
17:14 <tmortagne> Trefex: I think there is some scripts in http://extensions.xwiki.org to import all users but never used them myself
17:14 <vmassol> so the zipexplorer could use it fine
17:14 <vmassol> but it's no longer the case
17:14 <vmassol> so the real solution is a custom action for the zipexplorer plugin
17:14 <vmassol> so that it can parse its own url format
17:14 <vmassol> (which is something specific)
17:15 <tmortagne> Trefex: I guess it would be enough to import users once and then cound on active users to log enough for there profile to be up to date
17:15 <tmortagne> s/cound/count/
17:15 <Trefex> tmortagne: i just wanted to map the email field on existing users
17:15 <vmassol> gdelhumeau: how long would it be for 1.4M docs? :)
17:15 <vmassol> *large
17:16 <tmortagne> not sure there is up to date scripts to import uses actually
17:16 <Trefex> it's not to import, rather to sync existing ones
17:16 <Trefex> so foreach xwiki.user -> ldap sync
17:16 <tmortagne> yep but it's pretty much the same code
17:17 <Trefex> ok well then no gravatar for now :D
17:17 <tmortagne> it's not a very hard script to write
17:17 <tmortagne> sine the whole sync thing is a one method call
17:17 <tmortagne> all the script has to do is the foreach
17:17 <Trefex> that i can do
17:17 <Trefex> off to find the one method call
17:18 <gdelhumeau> it's around 350 bytes per document so ~4.56 Go
17:19 <tmortagne> Trefex: com.xpn.xwiki.plugin.ldap.XWikiLDAPUtils.updateUserFromLDAP
17:19 <tmortagne> you will have to use Groovy
17:19 <vmassol> gdelhumeau: I hope we don't need to read it all in memory inside xwiki or it won't work
17:19 <tmortagne> or any other real language (not Velocity basically)
17:19 <gdelhumeau> do you think we often delete a document having 1.4M children? (it's a real question)
17:20 <gdelhumeau> hmm and I think it's logged on the jobstatus anyway
17:20 <gdelhumeau> (in mean, in XWiki, not in the REST resource)
17:22 <vmassol> cjd: http://jira.xwiki.org/browse/XWIKI-12438?focusedCommentId=87491&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-87491
17:22 <cjd> sounds right
17:23 <cjd> so hax to make zipexplorer work and let everything else be broken
17:23 <vmassol> still pondering if I'll do a hax or do it properly with a custom action ;)
17:23 <vmassol> need more time to decide...
17:24 <tmortagne> has quit
17:29 <Trefex> this my first groovy script in xwiki, and getting
17:29 <Trefex> Caused by: javax.script.ScriptException: javax.script.ScriptException: groovy.lang.MissingMethodException: No signature of method: static com.xpn.xwiki.plugin.ldap.XWikiLDAPUtils.searchUserAttributesByUid() is applicable for argument types: (java.lang.String, [Ljava.lang.String;) values: [christophe.trefois, [mail]]
17:29 <Trefex> and the header in the javadoc states it should be (String, [] String) so no idea what's going on
17:31 <cjd> If you want to make code which stands the test of time, hack it in a few minutes with the full intent of replacing it :)
17:40 <tmortagne> has joined #xwiki
17:44 <ClemensR> has left #xwiki
17:46 <Ramona2> has quit
17:47 <Ramona2> has joined #xwiki
17:56 <Trefex> damn, this is tough. how do i get this sorted?
17:56 <Trefex> Caused by: javax.script.ScriptException: javax.script.ScriptException: groovy.lang.MissingMethodException: No signature of method: com.xpn.xwiki.plugin.ldap.XWikiLDAPConfig.getLDAPBindDN() is applicable for argument types: (com.xpn.xwiki.api.Context) values: [[email protected]]
17:58 <cjd> Without much knowledge of that particular function, I'm going to take a wild guess and suggest it requires an XWikiContext object, not a Context ?
17:58 <Trefex> cjd i have no clue how to get that
17:58 <cjd> in which case xcontext.getContext() will yield the object you are looking for
17:58 <Trefex> ah!
18:02 <Trefex> ah well i give up, don't understand shit :)
18:06 <evalica> has quit
18:06 <cjd> has quit
18:07 <Enygma`> vmassol: I guess you could do XWIKI-12205 by passing an XWiki.EntityReference to the constructor of XWiki.Document and then doing something like https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwiki-platform-web/src/main/webapp/resources/js/xwiki/create/create.js#L244 to create the URL.
18:07 <Enygma`> you`d also still have to support the old method signature for backwards compatibility
18:08 <vmassol> the whole thing has to be rewritten, it's not just for getting a url
18:08 <vmassol> (whole api using space and page)
18:08 <cjd> has joined #xwiki
18:08 <Enygma`> by "whole API", WDYM? XWiki.Document ?
18:09 <vmassol> yes and possible any other js api using space/page
18:09 <vmassol> *poosibly
18:09 <vmassol> *possibly
18:09 <Enygma`> XWiki.resource looks pretty pointless to me now, maybe we could retire it
18:10 <vmassol> right now the first decision to take si whether to go in the driection of the patch I put on the issue
18:10 <vmassol> yes the api is really strange
18:10 <vmassol> there are strange and redundant things in several places
18:11 <tmortagne> has quit
18:11 <vmassol> patch = either pass a serilalized ref or a json structure with the various ref elements
18:11 <Enygma`> iiih... a patch file... haven`t seen one of those in a while :)
18:11 <vmassol> in this patch, i've gone the serialized wy
18:11 <vmassol> *way
18:12 <vmassol> (right now I'm on the zipexplorer thing)
18:16 <KermitTheFragger> has quit
18:16 <Ramona2> has quit
18:16 <Ramona2> has joined #xwiki
18:22 <Trefex> vmassol: well i can't get it done, can't even connect to LDAP it seems, used https://github.com/xwiki-contrib/application-ldaptools/blob/master/src/main/resources/XWiki/SynchronizeLdapUsers.xml to inspire myself but it's in vein
18:22 <vmassol> sorry can't help on this. I'm also busy on something else atm
18:22 <Trefex> yeah no worries, just gonna abandon this
18:25 <cjd> has quit
18:38 <Ramona2> has quit
18:42 <tmortagne> has joined #xwiki
18:46 <Enygma`> vmassol: what about if for XWIKI-12372 we only disable the terminal document checkbox (that only appears for advanced users anyway), since it would be more explicit this way towards what is actually going on?
18:46 <Enygma`> (I actually already did this in a previous commit :) by "mistake" )
18:47 <tmortagne> has quit
18:48 <vmassol> Enygma`: by disable you mean it would be visible but not selectable?
18:48 <Enygma`> the "tocreate=space" parameter is still supported by the create action and the title says "Create Space", but the rest of the UI talks about pages anyway
18:48 <Enygma`> yes
18:48 <vmassol> what's the rationale of making it visible?
18:48 <vmassol> it has not meaning for a space AFAIK
18:48 <vmassol> *no
18:48 <vmassol> I'd rather we don't show it at all
18:48 <Enygma`> a space is nothing more than a non-terminal document... to an advanced user this would make sense
18:49 <vmassol> exactly so why show the option?
18:49 <vmassol> it has no meaning to me
18:49 <vmassol> and the rest of the UI is just wrong IMO
18:51 <Enygma`> so you want to add yet another layer of labels to the same UI to make it "look" like something else?
18:52 <Enygma`> I`m not sure what you are suggesting
18:52 <vmassol> I'm suggesting that showing the terminal page option when creating a space is plain wrong
18:53 <vmassol> it has no meaning at all
18:53 <Enygma`> I`ve just explained that it can be understood as a status or information on what you are about to create, not necessarily as an option, since that checkbox is checked or not depending on the action you are about to perform
18:53 <vmassol> what you're saying is that we should simply remove create space and only have create page even when the user wants to create a space which is going backward in term of usability
18:54 <vmassol> s/saying/hinting
18:54 <Enygma`> painting an apple orange does not make it an orange
18:54 <vmassol> a space is not a page
18:54 <vmassol> oterhwie we wouldbn't have spaces ...
18:54 <Enygma`> a non-terminal page is a space :)
18:54 <vmassol> no
18:54 <vmassol> otehrwise we wouldn't have spaces
18:55 <vmassol> (in the UI)
18:55 <Enygma`> then what does the create space action actually create?
18:56 <vmassol> you're confusing model and UI
18:56 <Enygma`> you could argue that you can create a space even with a terminal page, but we don`t expose that in the UI
18:57 <vmassol> I really don't get it why you want to make it hard for the user
18:57 <vmassol> (harder than it can be that is)
18:58 <Enygma`> so making them juggle 2 concepts instead of 1 and then going through the struggle of explaining them which is which is easier? :)
18:58 <Enygma`> anyway. I`ll do it the way you suggested. But the rest of the UI still talks about pages currently.
18:58 <Enygma`> (except for the title, of course)
18:59 <vmassol> well you get my POV, maybe others don't agree and would side with you :)
18:59 <vmassol> I agree that it's not normal that the rest of the UI talks about pages
19:00 <Enygma`> ATM we no longer really have a space creation UI, just a deprecated set of parameters in the create action
19:00 <vmassol> my POV is that we shouldn't expose the impl in the UI
19:00 <vmassol> (impl of spaces)
19:00 <Enygma`> if we want to change that, we could talk about it for 7.3M1 probably
19:00 <vmassol> and that it should be 2 concepts: Space and Page
19:01 <Enygma`> weren`t we doing Nested Documents in the UI?
19:01 <vmassol> or that comments = xobjects
19:01 <vmassol> etc
19:01 <vmassol> sure, we're talking about NS here not ND
19:02 <vmassol> (create space will not exist by default anymore in ND)
19:03 <OSIMasson> has quit
19:04 <vmassol> "Note that even though we'll remove the Add > Space menu entry in Flamingo, we should still probably continue supporting the create space template action for extension that might call it or simply for other skins which might want to create spaces since the Space concept still exist at the model level."
19:04 <vmassol> so in the default UI we won't see create space
19:07 <vmassol> so to summarize, the fact that a space is implemented as a page should remain an implementation details for me and not exposed in the UI as much as possible.
19:09 <vmassol> (Enygma`)
19:09 <Enygma`> so basically you are proposing that we use a parallel set of labels to apply in this case to the create UI
19:09 <vmassol> yes
19:09 <vmassol> and a different ui
19:10 <vmassol> there are commo parts but they're different uis
19:10 <Enygma`> why? (except for terminal page checkbox maybe)
19:10 <vmassol> creating a space for ex only requires a space name while creating a page requires space + page
19:11 <Enygma`> not really
19:11 <Enygma`> creating a space requires a parent space, a space identifier and perhaps a space title
19:11 <vmassol> yes with NS we cheat a bit
19:11 <vmassol> s/NS/ND
19:12 <vmassol> but they don't mean the same thing
19:12 <Enygma`> actually, the current implementation of the backwards compatible space handling is that, indeed, only top level spaces can be created
19:12 <vmassol> anyway we should reuse to the max the common elements
19:12 <vmassol> to avoid copy paste
19:14 <vmassol> note: we could imagine allowing to set rights before creating the page or space too for example
19:15 <vmassol> (not saying we should do it :))
19:19 <vmassol> bbiab
19:27 <abusenius_> has joined #xwiki
19:33 <Enygma`> vmassol: created XWIKI-12445
19:46 <gdelhumeau> has quit
19:53 <vmassol> ok thanks Enygma`
20:03 <mflorea> has quit
20:07 <Enygma`> has quit
20:43 <cjd> has joined #xwiki
20:56 <Trefex> has quit
21:54 <Slashman> has quit
23:22 <vmassol> has quit
Tags:
   

Get Connected