Skip to content

Fix for Laravel Breeze auth

Pre-release
Pre-release
Compare
Choose a tag to compare
@xaviqv xaviqv released this 28 Feb 18:47

Bug fixes

  • PanicHD in a Laravel app with an auth package that uses App\Models\User gives an error in PanicHD\Member model
    • Affected PanicHD installations:
      • Laravel 8 new installations with laravel/breeze or other auth packages
      • Earlier Laravel versions not using laravel/ui
    • Patch:
      • Creation of serveral PanicHD\Member models
        • Member_AppUser that extends App\User
        • Member_AppModelsUser that extends App\Models\User
      • Creation of MemberTrait that holds all Member models functionality
      • In service provider, if Member model setting is not configured, check for App\Models\User existence
      • use App\User if previous model was not found
      • Solves #28. Thank you @wertex !
  • Routes
    • PanicHD was using "dashboard" route name, which is already used and overlapped by Laravel. Renamed to "panichd.dashboard"
  • Seeders
    • Demo seeder was giving an error on execution
  • Search form
    • Got hang when using "attachment_name" or "any_text_field" fields
    • Using any of these fields added several attachment table joins which was making the search too much slow
    • Now attachment search for both fields is done apart and added as a ticket id array to the main ticket search query, which is faster
  • Ticket List
    • Agent change modal window
      • Check to change status was not showing default status name
      • After clicking on submit button, modal window was not closing

Patches

  • Documentation
    • Package requirements
      • Added MySQL off "strict mode" requirement. I did encounter this upgrading to MySQL 8, because DataTables was throwing a GROUP BY error until I changed these settings. 
    • Installation process steps reviewed and clarified
  • Summernote config file
    • Check file existence before loading it
  • Ticket complete / reopen action
    • Was deleting last complete / reopen comment, despite if last action was done by other member of if there was any comment between last comment and current action
    • Now it is only deleted if last comment of this kind is done by the same member and within 1 minute
  • Search form
    • "attachment_name" field has been changed to "attachment_text"

New settings

  • embedded_image_prefix : Name prefix for embedded images above resolution limit, which get transformed into an attachment