Re: [evolution-patches] Patch to allow a contact to be opened in the editor from command-line.

[Not Zed <notzed ximian com>]

On Mon, 2005-07-11 at 15:42 +0530, Veerapuram Varadhan wrote:
> On Mon, 2005-07-11 at 15:32 +0800, Not Zed wrote:
> > You shouldn't do anything AFTER the parent dispose method is
> invoked.
> > 
> I will correct it.  Thanks.
> 
> > Why is it done as query strings, rather than uri parameters like the
> > mailer?  Why can't you specify the source uid as part of the uri
> > specification (i.e. host/usrname) rather than as a query parameter?
> > 
> I just followed the existing URI handling in calendars.  Well,
> source-uid and contact-uids' can be passed as parameters.  However, I
> am
> not sure how to pass them as part of uri specification (like
> host/username etc). 

Easily.

url = url_new();
url.set_parameter("name", "value");

etc.

NEVER.  And I mean NEVER, EVER, use printf to create url strings.

> I used them as "query" strings as they sounded like "querying" for a
> contact and opening it.  Let me know the right way to follow, I am
> sure
> I will correct it.
> 
> > The query decoding is too simple too, it needs to do proper
> > url-decoding
> > of the parameters, if you're going to go that route - which should
> be
> > avoided anyway (simply because you need to do so much more work than
> > te
> > url parameters already do for you).
> > 
> Yes, I agree, but, the "EUri" already decodes them so, we might not
> need
> another decoding after that.
> 
> Let me know if I am wrong. 

How can it possibly do that? If you have a '&' encoded in the parameter
value, it will have to be %XX encoded separately to the &