Automattic / Automattic/jetpack

Pods conflict with Markdown loader

Open
#1,338 7 comments 0 reactions 0 assignees View on GitHub
[Feature] Markdown [Focus] Compatibility [Pri] Low Bug Good For Community
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.