patch 4 Auth/OpenID/Consumer.php

[Niels Berkers]

Jonathan Daugherty wrote:
> # querying the openid server gives a problem in
> # Auth/OpenID/Consumer.php A second call on function addExtensionArg
> # will over write the previous set argument. So the URL will only
> # contain a query for the last call (per $key * $namespace that is)
> 
> This is by design.  The method is intended to be called only *once*
> for required parameters, and only once for optional parameters.  If
> you use the OpenID 2 library, there is an entire module dedicated to
> the use of Simple Registration which is more straightforward than the
> API supplied by the 1.x.x libraries.
> 
uhmmm... ok (the specs claimed CSV's) But if there is a newer version of 
this library i will use that one... np here. Could give me a pointer 
where i can get it?

Just did an other little patch in case someone still wants to use it :)
It checks for optional and required. so you can use the function to set 
the policy_url
----------------------
  function addExtensionArg($namespace, $key, $value)
     {
         $arg_name = implode('.', array('openid', $namespace, $key));
         if ((count($this->extra_args[$arg_name]) > 0) && ($key == 
"optional" || $key == "required")){
                 $this->extra_args[$arg_name] .= ",".$value;
         } else {
                 $this->extra_args[$arg_name] = $value;
         }
     }
----------------------------------

br,

Niels Berkers
Quotar Internet & new media
http://openid.quotar.com