- last update
about 1 year ago Patch Failed to Apply - last update
about 1 year ago Patch Failed to Apply - last update
about 1 year ago Patch Failed to Apply - π¬π·Greece bserem
Patch #5 does work. #8 does not. Given how much of a difference this makes to imported content readability, I'll push this forward.
- πΊπΈUnited States hongpong Philadelphia
+ if (\Drupal::moduleHandler()->moduleExists('filter') && $this->configuration['filter_autop']) { + $text = _filter_autop($text); + }
The snag here is it would silently fail if filter module is not enabled. imo better if it would at least kick a notification. Or a better notice on the option like "Filter module must be enabled, or this step will be skipped silently." Some tweak of that nature. That's the only issue which jumps at me right now. Thanks so much for hashing this out to work cleanly!
+ '#description' => 'Converts line breaks into <p> and <br> in an intelligent fashion.',
- Merge request !15Issue #2927510 by phjou, dripa, hongpong: Wordpress new line to paragraph conversion β (Open) created by hongpong
- πΊπΈUnited States hongpong Philadelphia
re roll and generate merge request now that β¨ Import permalink as URL paths RTBC is committed as noted above. I think we should just add 'filter' to the required modules to cover this issue.
- π©π°Denmark ressa Copenhagen
Increasing Priority, since it is placed under "Major issues" header in π± Plan for wordpress_migrate 8.x-3.x beta release Active , so I am also adding that as parent issue.
- πΊπΈUnited States hongpong Philadelphia
added note re getting pluggable nature refactor on this β¨ Support contrib wordpress data / custom data | Yoast SEO Needs review