----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/1980/#review3252 -----------------------------------------------------------
cli/lmi/shell/compat/CIMIndicationListener.py http://reviewboard-openlmi.rhcloud.com/r/1980/#comment1794
whole "if" clause could be replaced with
path = path.lstrip('/')
which would also catch multiple slashes
- Alois Mahdal
On Aug. 27, 2014, 3:01 p.m., Peter Hatina wrote:
This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/1980/
(Updated Aug. 27, 2014, 3:01 p.m.)
Review request for OpenLMI Developers.
Repository: openlmi-tools
Description
Fix indication delivery when using compat listener. The problem was caused by wrong prefix prefix present in the indication name.
Diffs
cli/lmi/shell/LMIMethod.py 885f251e4dd64de5261f32e7c538f02a0c55fd17 cli/lmi/shell/compat/CIMIndicationListener.py 6b8a2d18a5594925942ce79e282879f0e35f35ab
Diff: http://reviewboard-openlmi.rhcloud.com/r/1980/diff/
Testing
Tested using LMI_FileSystemConfigurationService.SyncLMI_CreateFileSystem(...)
Thanks,
Peter Hatina