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

OJS 3.4 upgrade returns error: Unrecognized DAO SectionDAO #85

Closed
israelcefrin opened this issue Jun 23, 2023 · 5 comments
Closed

OJS 3.4 upgrade returns error: Unrecognized DAO SectionDAO #85

israelcefrin opened this issue Jun 23, 2023 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@israelcefrin
Copy link
Contributor

israelcefrin commented Jun 23, 2023

Describe the bug
After upgrading an installation from OJS 3.3 to OJS 3.4 the journal stops to work on the frontend when Immersion Theme is enabled. It returns an error code 500

To Reproduce
Steps to reproduce the behavior:

  1. Upgrade OJS 3.3 to 3.4
  2. Go to dashboard
  3. Enable and select Immersion theme (using tag 1.1.2)
  4. Check the home page
  5. See the error

Expected behavior
Loading the public interface.

Desktop (please complete the following information):

  • OS: MacOS
  • Browser chrome
  • Version 115

Additional context
Apache log error message:

Fatal error: Uncaught Exception: Unrecognized DAO SectionDAO! in /var/www/html/lib/pkp/classes/db/DAORegistry.php:77 
Stack trace: 
#0 /var/www/html/plugins/themes/immersion/z(186): PKP\db\DAORegistry::getDAO('SectionDAO') 
#1 [internal function]: ImmersionThemePlugin->addIssueTemplateData('TemplateManager...', Array) 
#2 /var/www/html/lib/pkp/classes/plugins/Hook.php(140): call_user_func_array(Array, Array) 
#3 /var/www/html/lib/pkp/classes/plugins/Hook.php(113): PKP\plugins\Hook::run('TemplateManager...', Array) 
#4 /var/www/html/lib/pkp/classes/template/PKPTemplateManager.php(1300): PKP\plugins\Hook::call('TemplateManager...', Array) 
#5 /var/www/html/pages/index/IndexHandler.php(80): PKP\template\PKPTemplateManager->display('frontend/pages/...') 
#6 [internal function]: APP\pages\index\IndexHandler->index(Array, Object(APP\core\Request)) 
#7 /var/www/html/lib/pkp/classes/core/PKPRouter.php(334): call_user_func(Array, Array, Object(APP\core\Request)) 
#8 /var/www/html/lib/pkp/classes/core/PKPPageRouter.php(277): PKP\core\PKPRouter->_authorizeInitializeAndCallRequest(Array, Object(APP\core\Request), Array, false) 
#9 /var/www/html/lib/pkp/classes/core/Dispatcher.php(165): PKP\core\PKPPageRouter->route(Object(APP\core\Request)) 
#10 /var/www/html/lib/pkp/classes/core/PKPApplication.php(373): PKP\core\Dispatcher->dispatch(Object(APP\core\Request)) 
#11 /var/www/html/index.php(21): PKP\core\PKPApplication->execute() 
#12 {main} thrown in /var/www/html/lib/pkp/classes/db/DAORegistry.php on line 77
@touhidurabir
Copy link
Member

In 3.4.0, the SectionDAO has been removed and it has moved to the new Repository/DAO/Map pattern , check pkp/pkp-lib#7128 .

The core plugin file ImmersionThemePlugin.inc.php need some update regarding of replacing the old SectionDAO calls to new repository pattern implementation .

@touhidurabir touhidurabir added the bug Something isn't working label Jun 23, 2023
@touhidurabir touhidurabir self-assigned this Jun 23, 2023
touhidurabir added a commit to touhidurabir/pkp-immersion that referenced this issue Jun 23, 2023
@touhidurabir
Copy link
Member

@Vitaliy-1 PR to review
immersion --> #86

@Vitaliy-1
Copy link
Collaborator

Thanks, @touhidurabir!
I'm wondering, how did tests miss that... Was this SectionDAO refactoring merged after themes release for 3.4?

@israelcefrin
Copy link
Contributor Author

Thanks @touhidurabir ! I've patched my installation with the PR and now it is working perfectly.

Vitaliy-1 added a commit that referenced this issue Jun 27, 2023
#85 Replaced SectionDAO calls with repository method
@Vitaliy-1
Copy link
Collaborator

Merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants