Skip to content
GitLab
Verified Commit 75715470 authored by Côme Chilliet's avatar Côme Chilliet
Browse files

:ambulance: fix(sinaps) Fix acquisition when affectationPrincipal is an etablissement or is empty 

issue #5912
parent e408f0f4
dev 6245-adapt-the-ci-to-the-reorganisation-of-the-dev-tools-and-fixing-the-trigger-downstream-pipelines 6250-supann-configuration-backend-requires-account-life-cycle-section 6280-plugins-update-plugins-to-take-into-consideration-the-new-directory-of-core-structure 6310-tasks-reminder-error-in-the-schema-duplicate-attribute-id 6311-put-the-version-1-5-in-all-yaml-for-fusiondirectory-1-5 6322-template-issue-when-creating-a-template-with-empty-password-error-message-should-not-be-seen-2 6332-zimbra-allows-update-of-data-for-unknown-domain-name-for-specific-individual-aliases 6337-webservice-issue-with-archiving-post-request-not-responding-but-successfully-archiving-user 6341-supann-extract-resources-states-sub-states-from-backend-configuration-to-their-own-objects 6346-lifecycle-adaptation-to-select-supann-resources-and-related-states-via-regex-mechanism master fusiondirectory-1.5 fusiondirectory-1.4 fusiondirectory-1.3.1
No related merge requests found
Hide whitespace changes
Inline Side-by-side
Showing
with 5 additions and 4 deletions
+5 -4
sinaps/include/class_sinapsRequestAcquisition.inc
+ 3
− 2
View file @ 75715470
...@@ -81,9 +81,10 @@ class sinapsRequestAcquisition ...@@ -81,9 +81,10 @@ class sinapsRequestAcquisition
/* rattachements */ /* rattachements */
if (isset($user['supannEntiteAffectationPrincipale'])) { if (isset($user['supannEntiteAffectationPrincipale'])) {
$personne->rattachements->rattachement->identifiantStructureDeRattachement = $codeEntiteToldapUuidCallback($user['supannEntiteAffectationPrincipale']); $personne->rattachements->rattachement->identifiantStructureDeRattachement = $codeEntiteToldapUuidCallback($user['supannEntiteAffectationPrincipale']);
$personne->rattachements->rattachement->roles->role->typeRole = 'EXT';
$personne->rattachements->rattachement->roles->role->externe->typeExterne = $typeExterne;
} }
$personne->rattachements->rattachement->roles->role->typeRole = 'EXT';
$personne->rattachements->rattachement->roles->role->externe->typeExterne = $typeExterne;
} }
public function getXML() public function getXML()
......
sinaps/personal/sinaps/class_sinapsUser.inc
+ 2
− 2
View file @ 75715470
...@@ -73,8 +73,8 @@ class sinapsUser extends simplePlugin ...@@ -73,8 +73,8 @@ class sinapsUser extends simplePlugin
* This is done here to make sure sinapsRequestAcquisition does not perform LDAP searches by itself * This is done here to make sure sinapsRequestAcquisition does not perform LDAP searches by itself
* */ * */
$entites = objects::ls( $entites = objects::ls(
'entite', array('entite','etablissement'),
array('supannRefId' => '*', 'supannTypeEntite' => 1, 'dn' => 'raw'), array('supannRefId' => '*', 'dn' => 'raw'),
NULL, NULL,
'(supannCodeEntite='.$codeEntite.')' '(supannCodeEntite='.$codeEntite.')'
); );
......
  • Ghost User @ghost ·

    SonarQube analysis reported 3 issues

    • :warning: 3 major

    Note: The following issues were found on lines that were not modified in the commit. Because these issues can't be reported as line comments, they are summarized here:

    1. :warning: This function "getUserValues" has 198 lines, which is greater than the 150 lines authorized. Split it into smaller functions. :blue_book:
    2. :warning: Remove this unused "$class" local variable. :blue_book:
    3. :warning: Review the data-flow - use of uninitialized value. :blue_book:
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment