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

Broken entry/sense properties #1717

Closed
myieye opened this issue Jan 27, 2023 · 1 comment · Fixed by #1718
Closed

Broken entry/sense properties #1717

myieye opened this issue Jan 27, 2023 · 1 comment · Fixed by #1718
Assignees
Labels
bug An existing problem with our app in production

Comments

@myieye
Copy link
Collaborator

myieye commented Jan 27, 2023

Describe the bug

A couple entry properties are missing/not configured correctly:

LexEntryModel->entryImportResidue: (see comment)

The requested lazy property "entryImportResidue" is not defined in "Api\Model\Languageforge\Lexicon\LexEntryModel#000000007d9070140000000050e15fbb" line 40 /var/www/html/vendor/ocramius/lazy-property/src/LazyProperty/Exception/InvalidLazyProperty.php 
DeepDiffDecoder.php line 108, DeepDiffDecoder::getNextStep(Api\Model\Languageforge\Lexicon\LexEntryModel, entryImportResidue, value, Value)

DeepDiffDecoder.php line 67, DeepDiffDecoder::applySingleDiff(Api\Model\Languageforge\Lexicon\LexEntryModel, Api\Model\Shared\DeepDiff\EditedDiff)

LexEntryCommands.php line 144, DeepDiffDecoder::applyDeepDiff(Api\Model\Languageforge\Lexicon\LexEntryModel, Array)

Sf.php line 574, LexEntryCommands::updateEntry(63c54d77ee52b1d3fa0e44e2, Array, 63c538bb183d7be68281d35b)

LexSense->status:

Undefined property: Api\Model\Languageforge\Lexicon\LexMultiValue::$value line 167 /var/www/html/Api/Model/Shared/DeepDiff/DeepDiffDecoder.php 
DeepDiffDecoder.php line 67, DeepDiffDecoder::applySingleDiff(Api\Model\Languageforge\Lexicon\LexEntryModel, Api\Model\Shared\DeepDiff\EditedDiff)

LexEntryCommands.php line 144, DeepDiffDecoder::applyDeepDiff(Api\Model\Languageforge\Lexicon\LexEntryModel, Array)

Sf.php line 574, LexEntryCommands::updateEntry(63c54d77ee52b1d3fa0e44e2, Array, 63c538bb183d7be68281d35b)

To Reproduce

  1. Go to the editor
  2. Put a value in one of the above mentioned fields (senes->status is a dropdown, I only have "unknown", which doesn't seem to get saved, but it triggers the exception)
  3. Wait for ~5s
  4. 💥
@myieye myieye added the bug An existing problem with our app in production label Jan 27, 2023
@myieye myieye self-assigned this Jan 27, 2023
@myieye myieye moved this from To Do to Awaiting Review in Language Forge Classic Jan 27, 2023
@myieye myieye mentioned this issue Jan 27, 2023
4 tasks
@myieye
Copy link
Collaborator Author

myieye commented Jan 31, 2023

It turns out that the import-residue field probably shouldn't exist, so I've created a new issue for that and have removed that from the scope of this issue.

@github-project-automation github-project-automation bot moved this from Awaiting Review to Awaiting Testing (QA) in Language Forge Classic Jan 31, 2023
@myieye myieye moved this from Awaiting Testing (QA) to Awaiting PO sign off in Language Forge Classic Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An existing problem with our app in production
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant