prabhakar267 / library-management-system

:books: An automated library management system developed in Laravel 4.2 PHP MVC Framework
MIT License
360 stars 323 forks source link

update to laravel 5.6? #39

Closed thiagogomesverissimo closed 1 year ago

thiagogomesverissimo commented 6 years ago

just checking if are there interesting in this update...

prabhakar267 commented 6 years ago

Not sure about the benefits of migrating from 4.7 to 5.6 But if there is any thing we can gain, then yes, I will be more than happy about the migration 👍

thiagogomesverissimo commented 6 years ago

@prabhakar267,

I think the idea is to be more in consonance with PHP new features (and remove deprecated things)... I spent some time today to up your project in ubuntu 18.04 and php 7.2 because mcrypt :wolf:

However, I never used lavarel before 5 version, but I can try to update it.

prabhakar267 commented 6 years ago

Great Please go ahead, let me know if you need some help 😄

thiagogomesverissimo commented 5 years ago

Just to let you know, I am working in this upgrade:

https://github.com/thiagogomesverissimo/library-management-system/tree/upgrade-to-5.8

barkermn01 commented 3 years ago

I know this issue is defunct now as Laravel has been updated to 6.20 in commit fda8903c8a30e5634ac804b5175e9f9f42e3a46e but does this mean the dependency on mcrypt has / can been removed, as when I search the source code I only see mcrypt used in the config and it's only using a constant? @prabhakar267