Automattic / Automattic/jetpack
Pods conflict with Markdown loader
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
Pods uses the base Markdown PHP project, while Jetpack uses Markdown + Extra. The current class exists check only checks the extra-specific class.
https://github.com/Automattic/jetpack/blob/master/_inc/lib/markdown/0-load.php
`if ( ! class_exists( 'MarkdownExtra_Parser' ) )`
However, the code inside of extra.php doesn't check if the 'Markdown' function exists yet, or some of the other things already defined/declared like constant(s).
How best should we proceed? Pods loads Markdown before Jetpack, but Jetpack needs Markdown+Extra.
Contributor guide
Research direction
Start by tracing the Markdown loading logic in _inc/lib/markdown/0-load.php and the referenced extra.php, including the existing class, function, and constant checks. Reproduce the load order where Pods loads base Markdown before Jetpack, then define done as Jetpack loading Markdown+Extra without redeclaration or compatibility conflicts.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100