Re: profile picture url [PATCH]

2013-02-11 Thread Vieri
Hi, --- On Sat, 2/9/13, Vieri wrote: > Thanks for the great work. I'll test it on Monday. Just to confirm that I checked it today and it works great. Thanks again! Vieri

Re: profile picture url [PATCH]

2013-02-09 Thread Vieri
a.wag...@gmail.com wrote: From: seba.wag...@gmail.com Subject: Re: profile picture url [PATCH] To: user@openmeetings.apache.org Cc: rentor...@yahoo.com Date: Friday, February 8, 2013, 5:47 PM Revision: 1444265 2013/2/9 seba.wag...@gmail.com @Maxim: Nice work :) @Vieri: I have added som

Re: profile picture url [PATCH]

2013-02-08 Thread seba.wag...@gmail.com
Revision: 1444265 2013/2/9 seba.wag...@gmail.com > > @Maxim: Nice work :) > > @Vieri: I have added some logic so that the user attributes will by synced > with every login from ldap to the user profile (in case the parameter is > defined and in case the search query in ldap does return somethi

Re: profile picture url [PATCH]

2013-02-08 Thread seba.wag...@gmail.com
@Maxim: Nice work :) @Vieri: I have added some logic so that the user attributes will by synced with every login from ldap to the user profile (in case the parameter is defined and in case the search query in ldap does return something for that param). Please verify. Note: It is of couse a backdra

Re: profile picture url [PATCH]

2013-02-06 Thread Maxim Solodovnik
done: Committed revision 1443331 On Thu, Feb 7, 2013 at 1:02 PM, seba.wag...@gmail.com wrote: > ok, > > I will modify probably the other issues on server side then and let the > client side as-is. > > Sebastian > > > 2013/2/7 Maxim Solodovnik > >> Yes I have applied that, modify a little and f

Re: profile picture url [PATCH]

2013-02-06 Thread seba.wag...@gmail.com
ok, I will modify probably the other issues on server side then and let the client side as-is. Sebastian 2013/2/7 Maxim Solodovnik > Yes I have applied that, modify a little and found it should be applied to > 5 more places > I would like to generalize it and commit (most probably later today

Re: profile picture url [PATCH]

2013-02-06 Thread Maxim Solodovnik
Yes I have applied that, modify a little and found it should be applied to 5 more places I would like to generalize it and commit (most probably later today, still need to work on issue sorting) On Thu, Feb 7, 2013 at 12:48 PM, seba.wag...@gmail.com < seba.wag...@gmail.com> wrote: > @Maxim: Have

Re: profile picture url [PATCH]

2013-02-06 Thread seba.wag...@gmail.com
@Maxim: Have you been applying Vieri's patch locally already? I don't want to duplicate any efforts. Sebastian 2013/2/6 Maxim Solodovnik > the code is duplicated in several places > going to generalize it > > > On Wed, Feb 6, 2013 at 3:19 PM, Maxim Solodovnik wrote: > >> The only issue I

Re: profile picture url [PATCH]

2013-02-06 Thread Maxim Solodovnik
the code is duplicated in several places going to generalize it On Wed, Feb 6, 2013 at 3:19 PM, Maxim Solodovnik wrote: > The only issue I see: places other than profile like user list, chat etc. > I'll double check > > > On Wed, Feb 6, 2013 at 3:18 PM, Maxim Solodovnik wrote: > >> I'm alre

Re: profile picture url [PATCH]

2013-02-06 Thread Maxim Solodovnik
The only issue I see: places other than profile like user list, chat etc. I'll double check On Wed, Feb 6, 2013 at 3:18 PM, Maxim Solodovnik wrote: > I'm already checking and going to commit with my changes ... > > > On Wed, Feb 6, 2013 at 3:16 PM, seba.wag...@gmail.com < > seba.wag...@gmail.com

Re: profile picture url [PATCH]

2013-02-06 Thread Maxim Solodovnik
I'm already checking and going to commit with my changes ... On Wed, Feb 6, 2013 at 3:16 PM, seba.wag...@gmail.com wrote: > Hi Vieri, > > I will check the patch, but it might take a while. > The other issue would be also possible to solve, we do sync for example I > think the password by every

Re: profile picture url [PATCH]

2013-02-06 Thread seba.wag...@gmail.com
Hi Vieri, I will check the patch, but it might take a while. The other issue would be also possible to solve, we do sync for example I think the password by every login from ldap to our database (if configured as "sync_password" = true). We would have to do simply the same for every attribute. So

Re: profile picture url [PATCH]

2013-02-05 Thread Vieri
Hi, I'm attaching a patch to fix the picture URI in case it's HTTP in the dashboard. This should fix the first part of my report. However, the second problem is still there and is probably harder to fix (see below - previous email). Basically, once a user has logged in once via LDAP and is crea