Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Class Legal_Installer does not exist #30

Closed
shefik opened this issue Jan 10, 2014 · 6 comments
Closed

Class Legal_Installer does not exist #30

shefik opened this issue Jan 10, 2014 · 6 comments

Comments

@shefik
Copy link
Member

shefik commented Jan 10, 2014

Latest changes now throw this error:

Class Legal_Installer does not exist

@craigh
Copy link
Member

craigh commented Jan 10, 2014

when? doing what?

@shefik
Copy link
Member Author

shefik commented Jan 10, 2014

I already had the Legal module installed. Then, I just replaced it with the latest code from the repository. The first issue were the existing hooks that I needed to manually remove from the database. After that, I went to the Extensions page and saw "Files missing" indicated for the Legal module. Trying to ciick the trash cash to "uninstall" throws the error I mentioned.

@craigh
Copy link
Member

craigh commented Jan 10, 2014

These are core issues. The extensions module isn't handling the upgrade correctly.

@craigh craigh closed this as completed Jan 10, 2014
@craigh
Copy link
Member

craigh commented Jan 10, 2014

@shefik - Legal doesn't have any hooks. It does utilize "hook-like" events. That have been converted to a Dependency Injection mechanism now. I assume you attempted to upgrade Legal before upgrading to most recent core (there have been two recent commits that attempt to deal with these errors: zikula/core#1543 and zikula/core#1542). So hopefully the errors you mentioned will not break your site (there will still be errors logged until you manually delete the rows in the db - these are a result of the development process and wouldn't happen in a production environment.)

As for the the Error listed above: Class Legal_Installer does not exist. Please report this to the core, but please provide the entire stack trace with as much information as possible. I tried to look in to it, but don't have enough information to proceed.

@craigh
Copy link
Member

craigh commented Jan 10, 2014

could your error possibly by related to zikula/core#1278 ?

@shefik
Copy link
Member Author

shefik commented Jan 11, 2014

@craigh I just went ahead and did a clean install instead, so it all works fine via that method.

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

No branches or pull requests

2 participants