CO-2675_UnixCluster_plugin_does_not_use_Primary_Name_for_GECOS

[ioigoume]

No description provided.

[boshrin]

I'm confused by the need to change

'CoPersonRole' => array('CoPerson' => 'PrimaryName')

to

'CoPersonRole' => array('CoPerson' => array(
  'PrimaryName' => array('conditions' => array('PrimaryName.primary_name' => true))
))

PrimaryName is defined as a relation on both CoPerson and OrgIdentity so that related queries automatically pick up the conditions necessary to filter on primary_name=true without having to respecify them in each related model. Is something breaking that?

[ioigoume]

I'm confused by the need to change

'CoPersonRole' => array('CoPerson' => 'PrimaryName')

to

'CoPersonRole' => array('CoPerson' => array(
  'PrimaryName' => array('conditions' => array('PrimaryName.primary_name' => true))
))

PrimaryName is defined as a relation on both CoPerson and OrgIdentity so that related queries automatically pick up the conditions necessary to filter on primary_name=true without having to respecify them in each related model. Is something breaking that?

This Pull Request is related to PR-539 that has been already merged. I will retest/clean up the fix and follow up on the question/comment