Skip to Content.
Sympa Menu

comanage-dev - Re: [comanage-dev] today's comanage-dev call

Subject: COmanage Developers List

List archive

Re: [comanage-dev] today's comanage-dev call


Chronological Thread 
  • From: Marie Huynh <>
  • To: Benn Oshrin <>
  • Cc:
  • Subject: Re: [comanage-dev] today's comanage-dev call
  • Date: Mon, 17 Sep 2012 14:45:55 -0700

This is in delete() in Standard Controller:

    if(!$this->restful)
    {
      // Delete doesn't have a view, so we need to redirect back to index regardless of success
      
      if($this->requires_person)
        $this->checkPersonID("force", $op);
      else
        $this->performRedirect();
    }


performRedirect(); isn't necessarily run.  Is there a reason for this and does anything depend on this being the way it is?

On Fri, Sep 14, 2012 at 10:34 AM, Heather Flanagan <> wrote:
Some highlights from today's Dev call:

Dev business

    naming scheme for variables
        the variable in question is the one added to the model or
controller; there is no specific standard, but generally variables
should be readable and understandable and ok if they are a little bit longer
    CO-438 discussion
        see email thread to comanage-dev from Marie starting 10-Sep-2012
@ 11:53
        punch list for ticket:

    need tabified view for CO org people and edit tabs out of CO Person
Roles (https://coriander.phys.uwm.edu/gears/co_person_roles/edit/1/co:1);
    deleting an email address doesn't take you back to the same tab (but
it should);
    this view
https://coriander.phys.uwm.edu/gears/org_identities/edit/1/co:1/tab:email vs
that view
https://coriander.phys.uwm.edu/gears/co_people/edit/1/co:1/tab:email
(make the CO Person bar like the CO Org Person bars

    Keep the tiger stripes
    Keep the vertical white line
    The CO people groups tab requires the pencil icon




Archive powered by MHonArc 2.6.16.

Top of Page