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

Fatal error due to missing function Submission::setCoverImage in OMP #9811

Open
asmecher opened this issue Mar 20, 2024 · 2 comments
Open

Fatal error due to missing function Submission::setCoverImage in OMP #9811

asmecher opened this issue Mar 20, 2024 · 2 comments
Assignees
Labels
Bug:2:Moderate A bug that is causing problems for a substantial minority of users.
Milestone

Comments

@asmecher
Copy link
Member

asmecher commented Mar 20, 2024

Describe the bug

From https://forum.pkp.sfu.ca/t/cover-image-max-width-and-height/86955:

When trying to change the max width and height in OMP3.3.0.17 I get an error message, " An unexpected error has occurred. Please reload the page and try again." I see that there was a previous topic on this, but changing it in the db itself does not fix the problem.

What application are you using?
OMP version 3.3.0-17

Additional information

Searching the code for setCoverImage, there is no function by this name, and it is only called in OMP's classes/services/ContextService.inc.php class. Probably the code was partially modernized away from getters on the submission object, but this call was missed.

@asmecher asmecher added the Bug:2:Moderate A bug that is causing problems for a substantial minority of users. label Mar 20, 2024
@asmecher asmecher added this to the 3.3.0-18 milestone Mar 20, 2024
touhidurabir added a commit to touhidurabir/omp that referenced this issue Apr 17, 2024
@touhidurabir
Copy link
Member

@asmecher PR to review
omp --> pkp/omp#1546

I am unable to reproduce the issue in local dev with latest checkout of stable-3.3.0 and also in test drive from Website > Appearance > Advanced . However I can see that there is not such method as Submission:: setCoverImage and it should be removed . Also code should never reach to that place as for submissions, the cover image manipulation done before .

@asmecher
Copy link
Member Author

I think this might be compounded by bad data that wasn't migrated from submission_settings to publication_settings. I've followed up on the forum to confirm with the original reporter: https://forum.pkp.sfu.ca/t/cover-image-max-width-and-height/86955/8

@asmecher asmecher modified the milestones: 3.3.0-18, 3.3.0-19 Jul 12, 2024
@asmecher asmecher modified the milestones: 3.3.0-19, 3.3.0-20 Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug:2:Moderate A bug that is causing problems for a substantial minority of users.
Projects
None yet
Development

No branches or pull requests

2 participants