Yep, custom permissions is something I overlooked, nice catch Douglas. Whoever fix this please also update the unit test as this case was not exposed
Thanks George On Wednesday, August 17, 2011, Douglas Hubler <[email protected]> wrote: > I see the fixes to stream line the replication by using straight SQL > didn't account for custom permissions. It's late for me, so I'll send > this summary out in case someone has a change to look at it in the > morning. If not, I will pick it up then > > summary > =================== > > In file > sipXconfig/neoconf/src/org/sipfoundry/sipxconfig/admin/commserver/imdb/Permissions.java > > the QUERY var doesn't account for custom permissions. In the select > statement we need > , cv.value as perm_${N} > and in joins we need > left join setting_value cv on u.value_storage_id = > cv.value_storage_id AND cv.path='permission/call-handling/perm_${N}' > > where N cycles thru custom permission list. > > In addition, we need to read the custom permissions out of the result > set and ensure they get accounted for in XML. > > > On Tue, Aug 16, 2011 at 7:56 PM, Douglas Hubler <[email protected]> wrote: >> From what i can tell, custom permissions are not getting into >> /var/sipxdata/sipdb/permission.xml. I'm looking into it, not obvious >> from the commits. >> > _______________________________________________ > sipx-dev mailing list > [email protected] > List Archive: http://list.sipfoundry.org/archive/sipx-dev/ >
_______________________________________________ sipx-dev mailing list [email protected] List Archive: http://list.sipfoundry.org/archive/sipx-dev/
