[BUGFIX] Aficher les catégories de profiles cibles lors de la duplica…

…tion d'une campagne (PIX-13265) #9484
1024pix · Jul 10, 2024 · 766dbb4 · 766dbb4
2 parents 0552ac7 + 73ec7b0
commit 766dbb4
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 12 deletions.
diff --git a/orga/app/components/campaign/create-form.js b/orga/app/components/campaign/create-form.js
@@ -9,14 +9,6 @@ export default class CreateForm extends Component {
   @service intl;
 
   @tracked wantIdPix = Boolean(this.args.campaign.idPixLabel);
-  @tracked targetProfiles = [];
-
-  constructor() {
-    super(...arguments);
-    Promise.resolve(this.args.targetProfiles).then((targetProfiles) => {
-      this.targetProfiles = targetProfiles;
-    });
-  }
 
   get isMultipleSendingAssessmentEnabled() {
     return this.currentUser.prescriber.enableMultipleSendingAssessment;
@@ -27,7 +19,7 @@ export default class CreateForm extends Component {
   }
 
   get targetOwnerOptions() {
-    const options = this.targetProfiles.map((targetProfile) => {
+    const options = this.args.targetProfiles.map((targetProfile) => {
       return {
         value: targetProfile.id,
         label: targetProfile.name,
@@ -94,7 +86,7 @@ export default class CreateForm extends Component {
 
   @action
   selectTargetProfile(targetProfileId) {
-    this.args.campaign.targetProfile = this.targetProfiles.find(
+    this.args.campaign.targetProfile = this.args.targetProfiles.find(
       (targetProfile) => targetProfile.id === targetProfileId,
     );
   }

diff --git a/orga/app/routes/authenticated/campaigns/new.js b/orga/app/routes/authenticated/campaigns/new.js
@@ -1,6 +1,7 @@
 import Route from '@ember/routing/route';
 import { service } from '@ember/service';
 import pick from 'lodash/pick';
+import RSVP from 'rsvp';
 
 export default class NewRoute extends Route {
   @service currentUser;
@@ -45,15 +46,15 @@ export default class NewRoute extends Route {
       }
     }
 
-    return {
+    return RSVP.hash({
       campaign: this.store.createRecord('campaign', {
         organization,
         ownerId: this.currentUser.prescriber.id,
         ...(campaignAttributes ?? campaignAttributes),
       }),
       targetProfiles: organization.targetProfiles,
       membersSortedByFullName,
-    };
+    });
   }
 
   resetController(controller, isExiting) {

diff --git a/orga/tests/unit/routes/authenticated/campaigns/new_test.js b/orga/tests/unit/routes/authenticated/campaigns/new_test.js
@@ -91,6 +91,7 @@ module('Unit | Route | authenticated/campaigns/new', function (hooks) {
 
         const organization = EmberObject.create({
           id: 12345,
+          targetProfiles: new Promise((resolve) => resolve([])),
         });
 
         class CurrentUserStub extends Service {
@@ -142,6 +143,7 @@ module('Unit | Route | authenticated/campaigns/new', function (hooks) {
 
         assert.strictEqual(await model.campaign, duplicatedCampaignRecord);
         sinon.assert.calledWithExactly(createRecordStub, 'campaign', expectedCampaignAttributes);
+        assert.false(model.targetProfiles instanceof Promise);
       });
     });