vlados / laravel-unique-urls Goto Github PK

Discussed in https://github.com/vlados/laravel-unique-urls/discussions/47

Currently in the migration the slug is using string() function which is setting the column to varchar(255)
Needs to be changed to text() which not have limit

Create a new command in the package to generate all the urls for all models
Should have options:
--model= Specify only a model for which to execute the command
--only-missing Skip existing urls
--fresh Truncate table urls and generate fresh for every model

Scenario:

1. Create a new PageModel with title=New Page
2. Update model and set title=Test Page
3. Update model and set title=Something new
4. Update model and set title=Latest changes

This will create in the database multiple redirects to the page

Current behaviour

If the user go to /test-page he will be redirected multiple times (/test-page > /something-new > /latest-changes)

Expected behaviour

If the user go to /test-page he should be redirected only once times (/test-page > /latest-changes) or new-page > latest-changes

Create a new command in the package to check:

• If models have implemented correctly the traits and the needed functions
• If all urls in the database have correct handler class and function

Should have options:

• --model= Specify only a model for which to execute the command

If the model don't have urls in the database the method generateUrlOnUpdate() will not create this records

add an exception if the model has attribute or column called url