chore: Remove dead catid column from locations table #58

New Issue

2026-06-28T15:36:23Z

jmiller commented

2026-06-28 15:36:23 +00:00

Summary

The #__mokosuitestorelocator_locations table has a catid INT column that is no longer used. Multi-category support (#1) uses the junction table #__mokosuitestorelocator_location_categories instead.

Impact

Column is populated on insert but never read for filtering
Admin form still has a single-category dropdown bound to catid
No functional impact — just dead schema

Fix

Remove catid column from install.mysql.sql
Add SQL update file to DROP COLUMN catid
Remove catid field from location.xml form if still present
Clean up any model references

Priority

P3 — cleanup, no functional impact.

## Summary The `#__mokosuitestorelocator_locations` table has a `catid` INT column that is no longer used. Multi-category support (#1) uses the junction table `#__mokosuitestorelocator_location_categories` instead. ## Impact - Column is populated on insert but never read for filtering - Admin form still has a single-category dropdown bound to `catid` - No functional impact — just dead schema ## Fix 1. Remove `catid` column from `install.mysql.sql` 2. Add SQL update file to `DROP COLUMN catid` 3. Remove `catid` field from `location.xml` form if still present 4. Clean up any model references ## Priority P3 — cleanup, no functional impact.

jmiller commented

2026-06-28 15:51:27 +00:00

Fixed:

Removed catid column and idx_catid index from install.mysql.sql
Added sql/updates/mysql/01.00.02.sql with DROP COLUMN IF EXISTS catid and DROP INDEX IF EXISTS idx_catid
No model/form references to clean up — location form already uses multi-select junction table (categories field)

Fixed: - Removed `catid` column and `idx_catid` index from `install.mysql.sql` - Added `sql/updates/mysql/01.00.02.sql` with `DROP COLUMN IF EXISTS catid` and `DROP INDEX IF EXISTS idx_catid` - No model/form references to clean up — location form already uses multi-select junction table (`categories` field)

jmiller closed this issue

2026-06-28 15:51:56 +00:00

jmiller referenced this issue from a commit

2026-06-28 16:22:52 +00:00

fix: resolve all open issues — detail map, clustering, CSP, GROUP BY, cleanup (#34 #57 #58 #59 #60 #61)

jmiller referenced this issue

2026-06-28 18:49:29 +00:00

v1.2.0 — Categories, API, hardening, polish #62

Sign in to join this conversation.