indieweb / indieweb/wordpress-micropub
Plugin always adds e-content div to post content. Should it?
Open
- Dominant language
- PHP
- Stars
- 53
- Forks
- 12
- Avg merge
- 1d 5h
- Merged PRs (30d)
- 7
Description
Hi @dshanske et al! I'm seeing an odd behavior with HTML content: two `
`s, one inside the other. I assume that's not expected? For example, this JSON request:
```
```json
{
"type": ["h-entry"],
"properties": {
"name": ["foo"],
"category": ["reply"],
"content": [{"html": "well ok is that so http://bar.org"}],
}
}
```
results in this HTML:
```html
```
I'm guessing because of this code?
https://github.com/indieweb/wordpress-micropub/blob/9bccb0bdc501a1dca9441b713811248e82f43191/includes/class-micropub-render.php#L96-L100
`e-content` is probably up to the theme, right? Should the plugin be injecting this extra `e-content` div?
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in includes/class-micropub-render.php at the linked lines and trace how the supplied content becomes the rendered post HTML. Compare the plugin's e-content wrapper with the theme-generated markup in the example, then verify the expected ownership of that wrapper. Done means the duplicate e-content behavior is resolved or its intentional behavior is documented and covered by the relevant existing checks.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php, wordpress
- Domain
- api, backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100