COmanage Developers List

[Benn Oshrin <>]

So it looks like I started to implement a basic UI over the new 
attribute collection. This isn't a bug so much as a partial commit to a 
much more complex feature.

Basically, all of the new tables don't impact anything *once a person 
has been added*, so if you manually update the old tables the way they 
used to work (eg: simple inserts into cm_org_identities) you should be fine.

I've got to think a bit more about what I was trying to accomplish here 
before I decide on what to do with the code.

On 11/11/11 9:39 AM, Marie Huynh wrote:
> Sorry, I meant the org/add.  I installed php5-xsl, added it to the
> wiki, and promptly forgot about it. This is what I'm seeing:
>
> https://img.skitch.com/20111111-xs85tenmac525jnde9xumee5q2.jpg
>
> It looks like $cmp_ef_attribute_order is empty so the whole foreach is
> skipped in app/views/org_identities/fields.inc.  Will this page not
> work without an enrollment flow already set up?
> gears/co_enrollment_flows is blank.  I can add what I need directly to
> the database until it's fixed, but I'd rather do it through the UI
> until I'm more familiar with what's supposed to go in the new tables
> so I don't screw it up and wonder why things aren't working.
>
> M
>
>
> On Thu, Nov 10, 2011 at 3:17 PM, Benn 
> Oshrin<>
>   wrote:
> > Which part? php5-xsl or org/add? If the latter, I started and then got
> > caught up in another bug preventing me from looking at it. I hate nested
> > bugs. Gross.
> >
> > On 11/11/11 3:58 AM, Marie Huynh wrote:
> > > Hi Benn,
> > >
> > > Did you get a chance to take a look at this?
> > >
> > > Marie
> > >
> > > On Thu, Nov 3, 2011 at 5:59 PM, Benn 
> > > Oshrin<>
> > >     wrote:
> > > > Wait, did I add the php5-xsl requirement?
> > > >
> > > > It's totally plausible that I broke org_identities/add. I'll take a look
> > > > and
> > > > perhaps temporarily commit a revert. However, it's all going to get
> > > > rewritten in the not too distant future. Next Sunday, AD.
> > > >
> > > > -Benn-
> > > >
> > > > On 11/2/11 5:24 PM, Marie Huynh wrote:
> > > > > Just a heads up, this version requires php5-xsl.
> > > > >
> > > > > On Wed, Nov 2, 2011 at 2:17 PM, Scott 
> > > > > Koranda<>
> > > > >   wrote:
> > > > > > > Are any of you seeing this or is it just me?  I did a fresh checkout
> > > > > > > of revision 115.
> > > > > > >
> > > > > > > https://img.skitch.com/20111102-n7er987r8atmkckn5kkxywb84e.jpg
> > > > > >
> > > > > > I haven't had a chance to look yet...will look tomorrow if not
> > > > > > sooner.
> > > > > >
> > > > > > Scott
> > > > > >
> > > > > > > On Tue, Nov 1, 2011 at 10:13 AM, Benn 
> > > > > > > Oshrin<>
> > > > > > >   wrote:
> > > > > > > > I just checked in changes for CO-180. There were a handful of
> > > > > > > > conflicts
> > > > > > > > that
> > > > > > > > I resolved, but I suspect one or two may have inadvertently reverted
> > > > > > > > your
> > > > > > > > previous changes. You might want to compare your recent commits
> > > > > > > > against
> > > > > > > > r111
> > > > > > > > (or the current revision).
> > > > > > > >
> > > > > > > > Apparently email on commit isn't working yet.
> > > > > > > >
> > > > > > > > -Benn-