Merge pull request #8166 from stopfstedt/rm-slice

removes a couple of calls to Array.slice() where they are not needed.
ilios · Sep 27, 2024 · 9452873 · 9452873
2 parents 26e80ab + 489eccf
commit 9452873
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/frontend/app/components/reports/subject/session.js b/packages/frontend/app/components/reports/subject/session.js
@@ -116,7 +116,7 @@ export default class ReportsSubjectSessionComponent extends Component {
           ? `${course.year} - ${course.year + 1}`
           : `${course.year}`,
         striptags(description),
-        striptags(mapBy(sessionObjectives.slice(), 'title').join()),
+        striptags(mapBy(sessionObjectives, 'title').join()),
       ];
     });
 

diff --git a/packages/frontend/app/controllers/curriculum-inventory-report/index.js b/packages/frontend/app/controllers/curriculum-inventory-report/index.js
@@ -27,7 +27,7 @@ export default class CurriculumInventoryReportIndexController extends Controller
     if (!this._sequenceBlocks) {
       return [];
     }
-    return this._sequenceBlocks.slice().filter((block) => !block.belongsTo('parent').id());
+    return this._sequenceBlocks.filter((block) => !block.belongsTo('parent').id());
   }
 
   @action