openedx / openedx/openedx-platform
Fix: DeprecationWarning: XBlocks should not instantiate their own field_data store during parse_xml()
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 8.2k
- Forks
- 4.4k
- Avg merge
- 6d 18h
- Merged PRs (30d)
- 42
Description
This warning is appearing 2 times during provisioning. It clutters logs and hinders dev experience (https://github.com/openedx/edx-platform/issues/32888).
It's being triggered in:
- video block: https://github.com/openedx/edx-platform/blob/e82016257280a9d9516187cc88a774b5fde04add/xmodule/video_block/video_block.py#L728
- xml block: https://github.com/openedx/edx-platform/blob/1aed4e66afba087abfc40e1234a672725d40555e/xmodule/xml_block.py#L370
AC
- The warnings stop cluttering logs
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing provisioning and observing the two warnings. Inspect the linked locations in xmodule/video_block/video_block.py around line 728 and xmodule/xml_block.py around line 370, focusing on parse_xml() and field_data store creation. Done means provisioning no longer emits these warnings and logs are no longer cluttered.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100