#wordpress 2022-06-12

2022-06-12 UTC
indierodo, jacky, V1, petermolnar and adamhorne joined the channel
#
adamhorne Hello?
#
david.shanske.com edited /Getting_Started_on_WordPress (+0) "/* WordPress out of the Box */" (view diff)
[snarfed] joined the channel
#
[snarfed] hey GWG, I tried using the Synd Links plugin to POSSE via Bridgy just now, and it's looking promising!...but I may have hit a bug. I unchecked the Syndicate To: Twitter via Bridgy checkbox, then published, and it triggered Bridgy Publish anyway. maybe because it adds the brid.gy synd link, and then the Webmentions plugin sends a wm to it?
#
GWG Unchecked? There's no option to check it by default.
#
GWG Just making sure I'm not confused.
#
GWG So, when you loaded the post, you checked it, then unchecked it? Or it was not checked, and it published anyway?
#
[snarfed] oh, it was unchecked
#
[snarfed] I also notice there are two u-syndication links to bridgy, one in .entry-meta > .entry-date and one in .entry-content > .syndication-links . not sure if that's expected, ok if so
#
GWG I've never seen that. I post with it all the time. Strange.
#
GWG The two links might be caused if somehow, the theme is adding the syndication links and so is the plugin
#
[snarfed] I did check the box and then uncheck it before publishing. probably unrelated, but still
#
GWG You did some custom code, didn't you?
#
GWG Maybe unexpected interaction
#
[snarfed] ah yeah that's my theme
#
[snarfed] oh, that's probably why then, synd links may prevent the wm to its own synd link, but not the one to the link injected by my theme?
#
GWG So, the plugin adds it, and to custom place it, you'd have to remove it from the_content filter
#
GWG If I changed anything since the last time you looked, it could have double-posted
#
GWG Either way, I have to dissect that code anyway, as I wanted to add a few options to debug
#
[snarfed] my concern is more, if the checkbox is unchecked, I don't want it trigger Bridgy Publish at all
#
[snarfed] if I remove synd links's synd link and keep my own, I assume the wm plugin will always trigger Bridgy Publish then, even if the checkbox is unchecked. right?
#
GWG [snarfed]: I agree 100%.
#
GWG It shouldn't, because the sending of the webmention to Brid.gy is done by Syndication Links based on the checkbox...
#
GWG But let me refamiliarize myself with how I configured it, as my memory might be off
#
GWG My theme custom places the links also, and I don't have this problem
#
GWG Can you post the custom code so I could compare?
#
[snarfed] yup, will do
#
[snarfed] https://gist.github.com/snarfed/e068890d95b0c1057fa93fd2edb9f2b2#file-functions-php-L258-L270=
#
GWG I'll explore every option.
#
[snarfed] example post: https://snarfed.org/2022-06-12_47059
#
[snarfed] thanks!
#
[snarfed] want me to file an issue?
#
Loqi [Ryan Barrett] Two powerful and opposing drives — the desire to enjoy the world and the urge to set the world straight. – E. B. White
#
GWG [snarfed]: Did you unhook the filter though?
#
[snarfed] I'll go ahead and do that
#
GWG I don't see it in the file.
#
GWG https://github.com/dshanske/syndication-links/blob/trunk/readme.txt#L82=
#
[snarfed] that's my current functions.php, I haven't modified it based on this conversation yet
#
GWG Yes, that might be the issue. Not unhooking the automatic function for non-supported themes
jacky joined the channel
#
[snarfed] ok! what's the filter to unhook?
#
GWG I just linked to it in the readme
#
GWG I just had an idea of giving it a specific hook, but I'll think about that in future
#
GWG A filter that returns true unless changed
#
[snarfed] got it, thanks! sorry for not seeing that in the docs 😁
#
[snarfed] oh, also, when I previewed the post (in WP) before I published it, I got a 500. I turned on debug logging in WP and reloaded the preview, and it rendered fine. 🤷
#
GWG Odd. Do the logs say anything?
#
GWG I need to figure out how to better test the cron initiated stuff
#
[snarfed] nothing relevant in the logs, sadly
#
[snarfed] this preview was a user-facing request, not cron-initiated
jacky joined the channel
#
GWG Two different thoughts, sorry
jacky, angelo, [aciccarello], jamietanna, jacky_ and Kaja_ joined the channel