Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

encodefromlegacy patch #4267

Closed
wants to merge 2 commits into from
Closed

encodefromlegacy patch #4267

wants to merge 2 commits into from

Conversation

renedekat
Copy link
Contributor

Fixed the encode_from_legacy method as it didn't decode properly. The mcrypt_decode method has changed from the 1.x version.
This update includes legacy methods for decode and mcrypt_decode. They are declare private.

Also added public/private to all methods and updated some PHPDoc

Fixed the encode_from_legacy method as it didn't decode properly. The mcrypt_decode method has changed from the 1.x version.
This update include legacy methods for decode and mcrypt_decode. They are declare private.

Also added public/private to all methods and updated some PHPDoc
@narfbg
Copy link
Contributor

narfbg commented Nov 25, 2015

CI2 is no longer supported, sorry.

@narfbg narfbg closed this Nov 25, 2015
@renedekat
Copy link
Contributor Author

Sure, but people might still want to migrate from older version and they'll run into the same issue as I did when upgrading an old project from 1.7 to 2.2.6, before updating to 3.x.

Perhaps you can merge it into the 3.x branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants