Skip to content
This repository was archived by the owner on Apr 7, 2021. It is now read-only.

Fix issue 30 and 28 #32

Closed
wants to merge 3 commits into from
Closed

Conversation

axelitus
Copy link

@axelitus axelitus commented Oct 8, 2018

Fixes broken package due to recent Laravel code changes explained in #28 and #30.

It even fixes the (apparently) broken tests:
screenshot-20181008100031-856x258

Fixes #28 and #30

This allows to replace phpunit configuration that should not be included
in the VCS to set custom environment variables.
getUnencryptedAttributes()

Laravel recently introduced a code change that broke the package as now
getAttributes() is called in performInsert() function instead of using
the attributes property directly.

Commit reference:
[laravel/framework@c540162]
@TechTailor
Copy link

Hey, thanks @axelitus for the fix. @austinheap would appreciate if you can merge the commits into the update.

@austinheap
Copy link
Owner

austinheap commented Nov 20, 2018

Closing in lieu of #30 (comment) / 3ef9a20. Thanks for all your help @axelitus!

@austinheap austinheap closed this Nov 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Laravel 5.7
3 participants