Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I think the appropriate approach for this is the pathologic module.
Comment | File | Size | Author |
---|---|---|---|
#14 | implement_pathologic-1829548-14.patch | 3.05 KB | mglaman |
#13 | implement_pathologic-1829548-13.patch | 2.17 KB | mglaman |
Comments
Comment #1
joelcollinsdc CreditAttribution: joelcollinsdc commentedattempted patch attached
Comment #2
populist CreditAttribution: populist commentedThere are a few problems we might run into with Linkit module..
1.) URLs are not properly aliased which the patch in #1 addresses
2.) Trouble with language support which #1793896: LinkitSearchPluginEntity needs createPath with language support addresses
Comment #3
populist CreditAttribution: populist commentedPostposing till a future release
Comment #4
populist CreditAttribution: populist commentedI think we might need the Pathologic module
Comment #5
ccarnnia CreditAttribution: ccarnnia commentedrerolled the patch
Comment #6
joelcollinsdc CreditAttribution: joelcollinsdc commentedapplying to both filters
Comment #7
populist CreditAttribution: populist commentedI am going to postpone since #1965864: Support CKEditor in Panopoly might shake this up.
Comment #8
populist CreditAttribution: populist commentedComment #9
PedroKTFC CreditAttribution: PedroKTFC commentedI'm trying panopoly. I use pathologic elsewhere and find it essential for sorting out image paths in dev/test/prod type uses. I've installed it but it doesn't seem to be working. Reading this thread I'm confused (not hard!). Am I right in thinking it shouldn't work with the standard distribution? Do I need the patch described here to get it to work with panopoly?
Comment #10
dsnopekThis patch adds the Pathologic filter to be used by the WYSIWYG text formats. You can also do that on your Drupal site and store the overrides in a module! So, this patch isn't strictly necessary to use Pathologic - it's to add Pathologic support by default, which probably is a good idea. I also use Pathologic on many of my sites. :-)
Comment #11
mglamanHere is a patch to implement Pathologic. Using Features Overrides isn't working all that great. For some reason the override for pathologic gets removed from the WYSIWYG text filter (but not HTML) whenever recreated.
Patch is from my duplicate #2290597: Add Pathologic for better LinkIt URLs
Comment #12
mglamanRe-roll of patch. Realized there was no install file to provide update hook which enables the Pathologic module! We also ran into a lot of URL issues by having the options set to "Full", patch now puts Pathologic at "Path" (relative) setting. This prevents issues where you have SSL/non-SSL or migrate domains.
Comment #13
mglamanRe-roll against latest dev.
Comment #14
mglamanRollin' rollin' rollin' to latest.
Comment #15
dsnopekComment #16
dsnopekActually, I think I added this to the sprint on accident :-)
Comment #17
mglamanWould this patch technically become postponed due to 1.x roadmap to maintenance and bug fixes? Wonder, because I'd like to write the tests for this, but don't want to if it's a no-go :)
Comment #18
dsnopekGiven that we already have so much other stuff to finish for Panopoly 1.x, I'm going to say "no" on this one and mark as Postponed.