Hi Peter,
No problem, moving the UriBuilder classes is one of the re-factoring items I’d
suggested (if it still makes sense) so I’ll take a look when I undertake that
task (I had hoped to get it done over Easter but clearly that didn’t happen!)
Thanks for all your efforts so far.
Regards,
Gav
Hi Gavin,
you can give it a try but then we have to write it in a generic way that
we can use it for all methods :)
Best regards,
Peter
On 4/21/14, 10:31 PM, "Gavin Cornwell" wrote:
>Hi Peter,
>
>Yep, the two properties on RepositoryInfo are not used currently and are
>actually stored in the
Hi Peter,
Yep, the two properties on RepositoryInfo are not used currently and are
actually stored in the binding session (as they are in the atompub binding) so
I would definitely suggest removing them.
Yes, I see the CMISURLUtil is used consistently in all methods, but we are
repeating the s
Hi Gavin,
thanks for your feedback. Yes those constants should be move to the
CMISBroserConstants header file.
Yup there should be rather a browser binding subclass for the
RepositoryInfo that holds those two properties. I will do that (or will
remove it as those properties are currently not used
Hi Peter,
You’ve made great progress already, I just have a few questions/comments about
the changes made in revision 1588209.
There are a couple of browser binding constants in CMISConstants.h
(kCMISParameterSelector & kCMISParameterSuccinct), can you please move these to
CMISBrowserConstants
