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

fix(subtypes): stop error on slots #1051

Merged
merged 1 commit into from
Sep 7, 2024
Merged

Conversation

ScuffedNewt
Copy link
Contributor

No description provided.

@SpeedyD
Copy link
Contributor

SpeedyD commented Sep 7, 2024

.....Um, sorry, but what does this actually fix? Seems more like it's removing functionality.

@ScuffedNewt
Copy link
Contributor Author

ScuffedNewt commented Sep 7, 2024

it fixes the slot tag
if you view the code the subtypes are treated differently further down, this code is unneeded since the subtype IDs are not contained in the character image


edit: adding some further clarification for anyone who may view this that I sent speedy in dms!
the subtype_ids should never have been present in the imageData because its not stored on the image table

for redesigns, the designUpdateManager puts the subtype_ids on the actual Request object, not on the image

further down in the code, there is an if(isset($data['subtype_ids']))... surrounding a forreach, this is the only needed part of the code

@ScuffedNewt
Copy link
Contributor Author

ScuffedNewt commented Sep 7, 2024

you can test to see, but I had a few big reports across multiple sites - this has fixed it
edit: example bug report
image
this shows that its the fact that subtype_ids does not exist on the image table that is causing the error

@SpeedyD
Copy link
Contributor

SpeedyD commented Sep 7, 2024

Right, fair enough. I think this error is primarily caused by the fact that that slot tags haven't been converted as did Design Updates and Character images, though new & edited slots shouldn't have this issue altogether.

On the other hand, the code does seem superfluous, as it's a relation now. It may be an issue for sites who have hard-coded a check for which image to use on subtypes, but that requires at most a check to fix.

@itinerare itinerare added bug Something isn't working reviewed Pull requests that have received community review and are pending merge labels Sep 7, 2024
@itinerare itinerare merged commit 0cf25a9 into corowne:develop Sep 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working reviewed Pull requests that have received community review and are pending merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants