okipa / laravel-model-json-storage Goto Github PK

Hi! Thanks for the tool! Testing currently, good job, but got one issue. So, now eloquent all() method is overriden by your lib and I can not use usual eloquent database-ment $chanels = Chanel::all(); to get all chanels from actual db, if I do not need json-storage?

One of the more appealing advantage of storing data in JSON rather than say sqlite is that it has potential to be properly versioned controlled via git. This cannot be leveraged upon as the current implementation stores the JSON data as a single line.

I would like to propose passing the JSON_PRETTY_PRINT constant in the toJson method during model saving so that it would pretty print the JSON. This would allow git to detect changed lines (single property inside the JSON) as opposed to replacing the whole collection should a merge conflict occur.

However this does increase the file size, if this is a concern, which it shouldn't be, you might want to expose this in the config file.

This would open up new possibilities and usecases for this package. Think of a team of devs working with a same set of versioned controlled JSON data without worrying much when a conflict occur.

Hi, thanks for the package, I was wondering if its possible to use relations like BelongTo from other models with this package?

Thank you for your time