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

Remove xp::errorAt() #299

Merged
merged 1 commit into from
Oct 21, 2021
Merged

Conversation

thekid
Copy link
Member

@thekid thekid commented Oct 21, 2021

This PR removes the xp::errorAt() function in favor of directly checking xp::$errors. This function is used in exactly one place outside of xp-framework/core:

zip/src/main/php/io/archive/zip/AbstractZipReaderImpl.class.php:145:      if (!\xp::errorAt(__FILE__, __LINE__ - 1)) {

thekid added a commit to xp-framework/zip that referenced this pull request Oct 21, 2021
@thekid
Copy link
Member Author

thekid commented Oct 21, 2021

This function is used in exactly one place

No longer, see https://github.com/xp-framework/zip/releases/tag/v9.0.1

@thekid thekid merged commit 8af0075 into xp-framework:main Oct 21, 2021
@thekid thekid deleted the refactor/remove-error-at branch October 21, 2021 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant