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

PEAR_Error is not implemented #528

Closed
liesauer opened this issue Sep 5, 2019 · 7 comments
Closed

PEAR_Error is not implemented #528

liesauer opened this issue Sep 5, 2019 · 7 comments

Comments

@liesauer
Copy link
Contributor

liesauer commented Sep 5, 2019

i try to compile ecshop project which is a famous online mall system(at least in china) and get this error:

/Users/imac/Projects/ecsharp/Website/ecshop/upload/includes/shopex_json.php(820,39): error PHP4044: Type name 'PEAR_Error' could not be resolved. [/Users/imac/Projects/ecsharp/Website/Website.msbuildproj]

btw, @jakubmisek do u mind opening a ECShop project?

@jakubmisek
Copy link
Member

PHP4044 indicated the class does not exist and is needed to compile the code properly. You can workaround it by commenting out the piece of code using PEAR_Error.

We don't have any plans to support PEAR since it is a deprecated functionality.

--

ECShop is a nice idea, however we don't have a plan to support it by ourselves - at least we can assist you with issues and fix what is necessary.

@jakubmisek
Copy link
Member

in WordPress we workaround PEAR_Error by commenting the code out

@liesauer we'll open ECShop project but with a note that we are not responsible :)

@liesauer
Copy link
Contributor Author

liesauer commented Sep 8, 2019

ok

@jakubmisek
Copy link
Member

test case:

class Err extends PEAR_Error {
}

@ethan-tqa
Copy link

ethan-tqa commented Sep 10, 2019

Got hit by the same error, but this time with latest Wordpress, version 5.2.3.
Edit: it's from wordpress\wp-includes\class-json.php (918,39)

@jakubmisek
Copy link
Member

yes, we are removing the use PEAR_Error from the sources for now

@jakubmisek
Copy link
Member

Closing the issue:

Workaround #1: add dummy PEAR_Error class into your code
Workaround #2: remove (Exclude) code that uses PEAR_Error since it is probably not being used anyways

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

No branches or pull requests

3 participants