Commit Graph

13 Commits (c086f20d3fd7e85c271ba2d95992387e4a062daa)

Author SHA1 Message Date
Your Name c0cde2f682 refactor: move 52 legacy models from App\ to App\Models\
Move all Eloquent models from the app/ root directory to app/Models/
for consistency with modern Laravel conventions. The project already had
54 models in App\Models; this migrates the remaining 52 legacy models.

Changes:
- Move 52 model files from app/ to app/Models/
- Update namespace declarations in each model
- Update all ~1000 import references across the codebase
- Add Relation::morphMap() in AppServiceProvider for backward
  compatibility with existing polymorphic database records
- Add missing HasSnowflakePrimary imports for models that relied
  on same-namespace resolution
4 weeks ago
Your Name c807a8524c refactor: replace short facade aliases with fully-qualified imports
Convert all 273 short facade alias imports (e.g. 'use Cache;') to their
fully-qualified class names (e.g. 'use Illuminate\Support\Facades\Cache;')
across 193 files.

This resolves 643 PHPStan 'class.notFound' errors caused by the static
analyzer being unable to resolve global aliases, and aligns with modern
Laravel conventions. It also unblocks removing the aliases array from
config/app.php in a future change.

All 107 tests pass.
1 month ago
Daniel Supernault d1799a0833
Lint commands 9 months ago
Christian Winther 2de9c70a0c
fix: don't restore memory limit after cities import
Since this command can only be invoked by CLI, the process will exit after a successful import, so restoring the transient PHP memory limit doesn't really have any affect.

In PHP 8.4, this throws the following error (which doesn't happen in 8.3 and below)

> [entrypoint / 11-first-time-setup.sh] - (stderr) 128769/128769 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%[2025-01-20 11:29:23] production.ERROR: Failed to set memory limit to 134217728 bytes (Current memory usage is 134746112 bytes) {"exception":"[object] (ErrorException(code: 0): Failed to set memory limit to 134217728 bytes (Current memory usage is 134746112 bytes) at /var/www/app/Console/Commands/ImportCities.php:140)

It seems to be a 8.4 behavior change, so removing the logic would make it go away
2 years ago
Dan Church c685e3e9f4
Fix global memory limit set in constructor
Instead, set it when running.

Having the set in the constructor was causing memory_limit to be changed
for "artisan queue:work" processes.
5 years ago
Daniel Supernault 91e58591be
Update ImportCities command, bump memory limit 7 years ago
Daniel Supernault d391deaf8a
Update ImportCities command, clean up 7 years ago
Daniel Supernault ea023fcf17
Update ImportCities command, check for corrupt or invalid checksum 7 years ago
Daniel Supernault 450602cbeb
Update Import Cities command, shorten country names 7 years ago
monkeyless 32a6100fed Add last progressbar advance 7 years ago
monkeyless 6d771819f4 Add checks back 7 years ago
monkeyless 27952883b6 Isert cities in chunks to increase performance 7 years ago
Daniel Supernault ae4af1675d
Add new commmand 7 years ago