comanage-dev - Re: [comanage-dev] r387 - registry/trunk/app/Controller (CO-494)
Subject: COmanage Developers List
List archive
- From: Benn Oshrin <>
- To:
- Subject: Re: [comanage-dev] r387 - registry/trunk/app/Controller (CO-494)
- Date: Tue, 23 Oct 2012 09:14:08 -0400
Fixed in r389. More comments below...
On 10/22/12 7:27 PM, Marie Huynh wrote:
I've not been able to reproduce this either. I suspect my system is
simply treating undefined variables as null and Scott's isn't.
That's an interesting thought. What's your value for error_reporting, and what version of PHP are you using?
There's a good write up at http://kunststube.net/isset which might lead to a revision of our coding style guide.
$email_addresses[0]['EmailAddress']['verified'] doesn't exist. Benn,
how is the verified field supposed to work?
It's set to 0/false/whatever until the invitation confirmation link is clicked.
However the problem here was that the form was broken for adding a new email address. It tried to reference $email_addresses, which will never be set on an add operation.
-Benn-
- [comanage-dev] r387 - registry/trunk/app/Controller, svnlog, 10/22/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller, Benn Oshrin, 10/22/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller, Scott Koranda, 10/22/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller, Marie Huynh, 10/22/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller (CO-494), Benn Oshrin, 10/23/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller (CO-494), Marie Huynh, 10/23/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller (CO-494), Benn Oshrin, 10/23/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller, Marie Huynh, 10/22/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller, Scott Koranda, 10/22/2012
- Re: [comanage-dev] r387 - registry/trunk/app/Controller, Benn Oshrin, 10/22/2012
Archive powered by MHonArc 2.6.16.