James Revillini
Forum Replies Created
-
I’ve created a temporary hotfix (version 3.6.1.1) that fixes both critical errors. The patched plugin is available for download and has been tested on production sites.
I’ve tested this but INSTALL AT YOUR OWN RISK. IT’S THE WEEKEND. I’M CHILLING AFTER THIS, NOT ANSWERING SUPPORT QUESTIONS LOL.
Download the Hotfix:
https://hirugaesu.com/post-smtp-3-6-1-1-temporary-hotfix-available/The hotfix includes:
- Fixed QRcode include path in
Postman/Mobile/mobile.php - Added null check for transport in
Postman/PostmanWpMail.php - Version bumped to 3.6.1.1 to distinguish from the broken 3.6.1
Important: This is a temporary hotfix until the official 3.6.2 release. The patched files will be overwritten if you update Post SMTP through WordPress, so you’ll need to reapply the patches or wait for the official fix.
Hi! I appreciate your consideration on this.
The tooltip idea works for me as long as it is showing the terms exactly as they were presented at the time of submission, not pulling from the current form data. This is helpful for a quick review, however it probably does not support users who must produce this information to a supervisor, legal department, or the client/contact, because the text of a tooltip is not going to be easily copied or printed.
The issue becomes more apparent if there are multiple terms & conditions fields presented in 1 form.
If you would like to continue keeping the admin UI clean, please consider adding a button to reveal ALL terms & conditions copy within the Entry view, as this will cover perhaps more use-cases.
Best Regards,
JimmyForum: Plugins
In reply to: [Debug This] WP Debug Log > Reset Log File → Page Not FoundWoop woop DT! 0.6.6 fixed the issue. Thank you!
I’m running
- RIO 1.8.4
- WP 6.7.2
- PHP 7.4.33
@aqibkhan9 Hey thank you! That works here!
Let me know if I can help any further!
Forum: Plugins
In reply to: [AnimatePress Free] [feature request] exclude links by selectorIn case anyone else needs to remove AnimatePress handling from certain elements, here’s one way (which may not work for your case, but it’s a start):
<script>
// REMOVE ANIMATEPRESS FROM FULLSCREEN POPS
document.addEventListener('DOMContentLoaded', () => {
setTimeout(function () {
// replace with whatever selector(s) works for you
document.querySelectorAll('.fstrigger').forEach(el => {
el.replaceWith(el.cloneNode(true));
});
}, 750);
});
</script>Forum: Plugins
In reply to: [AnimatePress Free] [feature request] exclude links by selectorI peeked at the JS for this plugin and I noted that you are attaching the click handlers at the link level. My suggestion is to alter this a bit to connect the event listener on the document level. This allows for more flexibility in attaching/removing/blocking event listeners and should improve overall efficiency.
Since my links that trigger popups also have the class
.fstrigger, I’ll use this to disable AnimatePress on these links.Forum: Plugins
In reply to: [Advanced Query Loop] Include Posts search not workingWhen I updated publish date on the pages that do not show to something more current (within 1 year), they start showing.
- This reply was modified 1 year, 3 months ago by James Revillini.
Forum: Plugins
In reply to: [Redirection] [code snippet] live-replace URLs before serving page→moved to gist https://gist.github.com/jrevillini/edf08952b2edb906f38644649099b33e
Go there for most up-to-date version 🙂
Forum: Plugins
In reply to: [Redirection] [code snippet] live-replace URLs before serving pageHere’s an updated version to take stabs at.
////////////////////////////////////////////////////////////////////////////////// //////////////////// 🚀 Schwifty-𝓪𝓼𝓼 Link Fixer 3000™ ///////////////////////////// ////////////////////////////////////////////////////////////////////////////////// // // function replace_old_links_in_content($content) { // Ensure Redirection plugin is active; if not, BAIL EARLY if (!is_plugin_active('redirection/redirection.php')) { return $content; } // Define allowed post types and specific post IDs (set to [] for all) $allowed_post_types = ['post']; // Modify for your use case $allowed_post_ids = [144]; // Set specific post IDs for testing or [] for all // Get the current post object global $post; if (!$post) { return $content; } // Check if the current post matches the post types or the allowed post ID if (!empty($allowed_post_types) && !in_array($post->post_type, $allowed_post_types)) { return $content; } if (!empty($allowed_post_ids) && !in_array($post->ID, (array) $allowed_post_ids)) { return $content; } // Extract all <a> tags from content if (preg_match_all('/<a\s[^>]*href=["\']([^"\']+)["\']/i', $content, $matches)) { $urls_to_check = array_unique($matches[1]); // Unique URLs only if (!$urls_to_check) { return $content; // No links to change, bail early } foreach ($urls_to_check as $old_url) { $redirects = Red_Item::get_for_url($old_url); // Returns an array of Red_Item objects if (empty($redirects) || ! is_array($redirects) || count($redirects) < 1 ) continue; foreach ($redirects as $redirect) { $action_data = $redirect->get_action_data(); if ( $action_data === '' || $redirect->is_regex() ) continue; // @TODO needs to be fixed to handle regex matching $new_url = esc_url( $action_data ); $replacements[$old_url] = $new_url; break; // Use the first valid redirect and move on } } // Only do a replacement if we found redirects if (!empty($replacements)) { foreach ($replacements as $old => $new) { $content = str_replace( 'href="' . $old . '"', 'href="' . $new . '" data-schwifty-oldhref="' . esc_attr($old) . '"', // keep old URL in data attribute cuz that's fun $content ); } } } return $content; } add_filter('the_content', 'replace_old_links_in_content', 20);Implementation:
- do not use on a production site
- try it using the Code Snippets plugin:
- create a new snippet, copy/paste this into the snippet with title INFRASTRUCTURE | LIVE-fix old URLs based on Redirection plugin rules , set it to run on front-end only, Save+Activate
- go to some pages that used to have old URLs – they should now be rewritten to the new/correct URLs
Known Flaws:
- Doesn’t work on regex matches yet
- Needs to work for relative URLs (this should work now but I haven’t tested it yet)
- May or may not work for links with parameters in them. I have no idea yet.
- If Redirection is being used to create short links and these are used in posts/pages, they’re all gonna get replaced and you’ll lose tracking (if that’s how you’re using Redirection)
- This reply was modified 1 year, 3 months ago by James Revillini.
- This reply was modified 1 year, 3 months ago by James Revillini.
Sorry for the late response … seems to be with latest WordPress, latest Admin Notices Manager, and PHP 8.1-8.3 … maybe this is a PHP module I need to enable?
Forum: Plugins
In reply to: [Redirection] Redirection doesn’t Work@strategik Maybe mark this thread resolved so it doesn’t appear to others that the plugin doesn’t work. This beast is one of the best pieces of work out there 😉
Yes, what @uriahs-victor said 🙂
+1
And thank you, @uriahs-victor for the tip on rolling back the version 🙂
In my case, emails coming from FluentCRM were affected, however some of the WP generated emails e.g. “XYZ Post is now published” did have the recipient in logs.
- This reply was modified 1 year, 3 months ago by James Revillini.
Forum: Plugins
In reply to: [Redirection] Redirection doesn’t WorkCan you provide more details, like a screenshot of your redirection for this URL?
Forum: Plugins
In reply to: [Advanced Query Loop] Include Posts search not workingI would love to try to troubleshoot this one too. In my case, I want to include just one regular page. Things I have in common with @ryantown ‘s setup:
- this is a page published quite awhile back – 2018 to be exact
- I have 1 CPT defined, but I’m not trying to find a CPT, just a regular page
- I can find other pages that are more recent, as well as recent posts
If it makes any difference, the page in question has child pages. Other pages that have children also do not come up, but I’m not sure if that’s a factor with this bug.
- Fixed QRcode include path in