@autobusal/utilities 1.4.0 → 1.4.1

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+
+## 1.4.1
+
+### Changed
+
+- **Gender can be "not declared".** `getOptionsSex` gains it as the first
+  option and `getSex` reports it, instead of treating everything that is not
+  `1` as male — which is what three existing rows with no value were already
+  rendering as.
+
+  Carried by `null`: the `visitors`.`sex` column has always been nullable, so
+  nothing migrated. The option's id is `''`, which Laravel's
+  `ConvertEmptyStringsToNull` turns into a real null on the way in, so "not
+  declared" is the *absence* of a value at every layer rather than a magic
+  number each one has to learn.
+
+  Deliberately not shared with the checkout passenger forms — those build
+  their own field, and gender there is a per-route requirement a carrier can
+  insist on. A profile is the person; a passenger record is what a carrier
+  demands in order to board them.
+
 ## 1.4.0
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/utilities",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts",

package/passengers.ts

@@ -1,15 +1,45 @@
 import { TFunction } from 'i18next';
 
-export const getSex = (value: number): string => (
-  value === 1 ? 'female' : 'male'
-);
+/**
+ * Edited: Ferjolt Ozuni - Date: 2026-08-06
+ *
+ * Gender is OPTIONAL on a profile, and "not declared" is a real answer
+ * rather than a missing one.
+ *
+ * `null` carries it - `visitors`.`sex` has always been nullable and already
+ * held rows with no value, so nothing needed migrating. What was missing was
+ * any way to SAY it: the select offered Male and Female, the profile form
+ * defaulted to Male, and anybody who never touched the field was recorded as
+ * male. Three such rows existed already and rendered as "male" here, because
+ * this treated everything that was not 1 as male.
+ *
+ * Deliberately NOT shared with the passenger forms at checkout: those build
+ * their own field (routes-order's Gender), and gender there is a per-route
+ * requirement a carrier can insist on. A profile is the person; a passenger
+ * record is what a carrier demands in order to board them, and those are
+ * different questions.
+ */
+export const getSex = (value?: number | null): string => {
+  if (value === null || value === undefined) {
+    return 'undeclared';
+  }
+
+  return value === 1 ? 'female' : 'male';
+};
 
 export const getOptionsSex = (t: TFunction<'common'>) => ([
   {
+    // Empty rather than a third numeric code: Laravel's
+    // ConvertEmptyStringsToNull turns it into a real null on the way in, so
+    // "not declared" is the ABSENCE of a value at every layer rather than a
+    // magic number each of them has to learn.
+    id: '',
+    name: t('data.gender.undeclared', { ns: 'common' })
+  }, {
     id: 0,
     name: t('data.gender.male', { ns: 'common' })
   }, {
     id: 1,
     name: t('data.gender.female', { ns: 'common' })
   }
-]);
+]);