search

statamic / eloquent-driver

Provides support for storing your Statamic data in a database, rather than flat files.
https://statamic.dev/tips/storing-content-in-a-database
MIT License
104 stars 74 forks source link

Errors: Access level to Statamic\Eloquent\Fields\BlueprintRepository must be protected #93

Closed nbeerten closed 1 year ago

nbeerten commented 1 year ago

I am getting errors at both php please eloquent:import-entries and just visting any page accessing data from an entry. I have been able to fix this myself by editing Statamic\Eloquent\Fields\BlueprintRepository and changing occurrences private in front of properties and methods to protected. This is the first error I got: 

Access level to Statamic\Eloquent\Fields\BlueprintRepository::BLINK_FOUND must be protected (as in class Statamic\Fields\BlueprintRepository) or weaker

Screenshot of error in console: image

ryanmitchell commented 1 year ago

Related to:Fatal error exception after upgrade of statamic/cms to version v3.3.57 · Issue #92 · statamic/eloquent-drivergithub.com

If you have the code changes required please open a PR

nbeerten commented 1 year ago

I'd rather not open a PR myself, both because I am not sure how to do it and because I am not sure if this is caused by a wrong setup on my side, or if the change could affect other parts of the code...

ryanmitchell commented 1 year ago

Its been caused by:

https://github.com/statamic/cms/pull/6588

If you dont get to it I'll work on the fixes required next week.

nbeerten commented 1 year ago

Alright, so the errors are caused by a fresh update I installed literally today... Was sort of expecting that already. Thanks in advance

jasonvarga commented 1 year ago

Duplicate of #92