Adding some code smell comments relating to #276. No code changes sin…

…ce it's the wrong branch.

src/helpers.mjs

@@ -65,6 +65,7 @@ export class Helpers {
      * @property {number} totalMinutes total elapsed minutes since 12am on day 0
      * @returns {import('./timekeeper.mjs').gameTurnTime}
      */
+    // todo: Code Smell! Why is this not a Timekeeper function? What architectural goal is served by having it here?
     static factorGameTurns (totalMinutes) {
         const gameTurnData = {}
         gameTurnData.totalGameTurns = Math.floor(totalMinutes / Constants.minutesPerStretch)
@@ -75,6 +76,7 @@ export class Helpers {
         gameTurnData.turns = remainingGameTurns % Constants.stretchesPerShift
 
         gameTurnData.shiftName = Helpers.getDragonbaneShiftName(gameTurnData.shifts)
+        // todo: Code Smell! should use Constants.daysPerWeek
         gameTurnData.day = { index: (gameTurnData.days % 7) + 1 } // 1-based day index for UI
         gameTurnData.day.name = Helpers.getWeekdayName(gameTurnData.day.index - 1) // lookup by 0-based index
         gameTurnData.weekNumber = Math.floor(gameTurnData.days / 7) + 1 // 1-based week number

src/timekeeper.mjs

@@ -212,8 +212,10 @@ export class Timekeeper {
         time.days = Math.floor(totalMinutes / Constants.minutesPerDay)
         time.hours = Math.floor((totalMinutes % Constants.minutesPerDay) / 60)
         time.minutes = (totalMinutes % Constants.minutesPerDay) % 60
+        // todo: Code Smell! should use Constants.daysPerWeek
         time.day = { index: (time.days % 7) + 1 } // 1-based day index for UI
         time.day.name = Helpers.getWeekdayName(time.day.index - 1) // lookup by 0-based index
+        // todo: Code Smell! should use Constants.daysPerWeek
         time.weekNumber = Math.floor(time.days / 7) + 1 // 1-based week number
 
         return time

src/uipanel.mjs

@@ -198,6 +198,7 @@ export class UIPanel extends HandlebarsApplicationMixin(ApplicationV2) {
                  */
                 id: 'etk-days',
                 value: time.day.index,
+                // todo: Code Smell! should use Constants.daysPerWeek
                 max: 7,
                 name: game.i18n.format('JDTIMEKEEPING.Time.DayAndWeek', {
                     day: time.day.name,