Mike Rockett Posted October 7, 2015 Author Share Posted October 7, 2015 Hi Guys, For some reason I cannot get JumpLinks 404 monitor to work, everything else is working and the CMS notifications show 404's but JumpLinks is not displaying the list of received 404's Anyone have an idea why this could be? I am running PW 2.6.1 Interesting... I just tested on 2.6.17 (one of my local testing installs) and it really isn't logging 404s... Let me have a quick look. Link to comment Share on other sites More sharing options...
Mike Rockett Posted October 7, 2015 Author Share Posted October 7, 2015 I have found the issue. If there are no Jumplinks registered scanAndRedirect simply returns false, which lands up skipping the process of logging the request. I have now fixed the issue both on master and dev. Will get Module Directory to see the update shortly. Thanks for the heads-up! Link to comment Share on other sites More sharing options...
quickjeff Posted October 7, 2015 Share Posted October 7, 2015 @Mike Rockett, After updating, I get this error: Parse error: syntax error, unexpected ''alphanum'' (T_CONSTANT_ENCAPSED_STRING), expecting ')' in /modules/ProcessJumplinks/ProcessJumplinks.module on line 92 Link to comment Share on other sites More sharing options...
Mike Rockett Posted October 7, 2015 Author Share Posted October 7, 2015 @Mike Rockett, After updating, I get this error: Parse error: syntax error, unexpected ''alphanum'' (T_CONSTANT_ENCAPSED_STRING), expecting ')' in /modules/ProcessJumplinks/ProcessJumplinks.module on line 92 Sorry about that - accidential input appeared on that line when attempting a grep. Issue has been resolved - please replace Jumplinks.module with the one from Github. 1 Link to comment Share on other sites More sharing options...
quickjeff Posted October 7, 2015 Share Posted October 7, 2015 @Mike Rockett - Thanks! Working good again! 1 Link to comment Share on other sites More sharing options...
Mike Rockett Posted October 8, 2015 Author Share Posted October 8, 2015 Have just bumped the dev branch up to 1.3.4, bringing several important updates to be merged with master soon. Firstly, and most importantly: In light of PW 2.7 being around the corner, the minimum PW version requirement is now 2.6.1. This allows the use of 2.6 goodies that were not used before. Mainly, Jumplinks now uses ModuleConfig. Next, the Entity Editor (the name used for the jumplinks editor page) is now working as expected. PageListSelect and PageAutocomplete are now syncing correctly. This is really just a visual gimmick for the purposes of consistency. The editor now looks much cleaner, and easier to use. Few other changes to note: Table font size has been increased a notch. Destination column now has page title and full path hint for jumplinks that reference existing pages. Table footer has been removed - I don't think this was very informative, and thus not needed. Please note that I have not tested everything extensively as yet - this will be done soon. As such, don't use this branch on a production site until merged with master. Once merged, I'll bump the version up to 1.5, considering the major changes/departures compared to the current 1.3.x. 1 Link to comment Share on other sites More sharing options...
Gazley Posted October 21, 2015 Share Posted October 21, 2015 Hi there, Is JumpLinks activated when a page is accessed via $pages->get($url)? I have some code calling a page using $pages->get() and the "old" url. The get() is returning a null page. The jumplink works if I access the page with the "old" URL in the browser address bar. Any advice appreciated! Many thanks. Link to comment Share on other sites More sharing options...
Mike Rockett Posted October 21, 2015 Author Share Posted October 21, 2015 (edited) Hi there, Is JumpLinks activated when a page is accessed via $pages->get($url)? I have some code calling a page using $pages->get() and the "old" url. The get() is returning a null page. The jumplink works if I access the page with the "old" URL in the browser address bar. Any advice appreciated! Many thanks. Hi Gazley, No, it only hooks to ProcessWire's 404 event, which fires only on HTTP requests that do not resolve to a Page. Were the old URIs from another CMS, or did you move pages around in an existing PW installation? Edited October 21, 2015 by Mike Rockett 1 Link to comment Share on other sites More sharing options...
Gazley Posted October 21, 2015 Share Posted October 21, 2015 Hi Mike - thanks for the confirmation! 1 Link to comment Share on other sites More sharing options...
Lance O. Posted November 13, 2015 Share Posted November 13, 2015 I am using Jumplinks for one of my clients. We have set up a number of redirects to monitor several campaigns we have in place. For instance, we have links like this: http://website.com/bioh that redirect to actual pages in the site: http://website.com/biohealth/ The original link is being used in Google AdWords. AdWords is adding a Google Click Identifier (gclid) to the url, so when a user clicks on the link, it actually appears like this when they visit the site: http://website.com/bioh?gclid=CPaHlL3_jckCFdc9gQod4xEB4g The problem is that the link with the gclid parameter is returning a 404 error. Jumplinks doesn't appear to recognize when a parameter is appended to the url. Is there a way to prevent this, or perhaps I didn't set something up correctly? Any advice is appreciated. I'm using Jumplinks 1.3.1 with ProcessWire 2.6.1. Link to comment Share on other sites More sharing options...
quickjeff Posted November 13, 2015 Share Posted November 13, 2015 You need to go into your GA account and change the Urls to the new Urls or you need to do this: http://www.website.com/bioh? gclid ={id:any} Redirects to http://www.website.com/biohealth?gclid ={id:any} 1 Link to comment Share on other sites More sharing options...
quickjeff Posted November 13, 2015 Share Posted November 13, 2015 @lance O. Also make sure you have the template setup to allow URL segments and remove the back slash. Especially if your using landing pages. Link to comment Share on other sites More sharing options...
Lance O. Posted November 13, 2015 Share Posted November 13, 2015 It looks like setting up the Source URL with the additional code works. (It doesn't look like we need to add the additional code to the page url.) Thank you for your quick response! Client is happy and has restored faith in PW and Jumplinks. 1 Link to comment Share on other sites More sharing options...
quickjeff Posted November 13, 2015 Share Posted November 13, 2015 @ Lance O. - No problem! I love Jumplinks and have used ProcessWire for tons of client projects! Have faith in it! 1 Link to comment Share on other sites More sharing options...
Mike Rockett Posted November 14, 2015 Author Share Posted November 14, 2015 I am using Jumplinks for one of my clients. We have set up a number of redirects to monitor several campaigns we have in place. For instance, we have links like this: http://website.com/bioh that redirect to actual pages in the site: http://website.com/biohealth/ The original link is being used in Google AdWords. AdWords is adding a Google Click Identifier (gclid) to the url, so when a user clicks on the link, it actually appears like this when they visit the site: http://website.com/bioh?gclid=CPaHlL3_jckCFdc9gQod4xEB4g The problem is that the link with the gclid parameter is returning a 404 error. Jumplinks doesn't appear to recognize when a parameter is appended to the url. Is there a way to prevent this, or perhaps I didn't set something up correctly? Any advice is appreciated. I'm using Jumplinks 1.3.1 with ProcessWire 2.6.1. Just as a follow-up on this: In a private discussion with Jeff last month, I noted that this scenario will be accomodated in the next version. However, if there are more Jumplinks users bumping into this issue, then I would want to make it part of 1.5. In the discussion, I mentioned there would either be a checkbox in the entity editor that allows you to make query strings optional, or that we could use square parenthises to mark them as optional, like this: bioh[?gclid={token:any}]. After a little thought on the matter, I think it may be better to use the checkbox approach for query strings. However, I think the behaviour should be somewhat different in that it would let you choose to "ignore the query string in source" and, if that is selected, "append original query string to destination". Thoughts? In light of the square parenthises idea, I also have plans to make parameters optional, like so: {?title} 1 Link to comment Share on other sites More sharing options...
quickjeff Posted November 14, 2015 Share Posted November 14, 2015 Hi Mike, I think the checkbox would be a great addition. This would make it easier for a newbie to Jumplinks be able to just dive in and get things done. I am for this update! 2 Link to comment Share on other sites More sharing options...
Mike Rockett Posted November 26, 2015 Author Share Posted November 26, 2015 @Mike Thanks for jumplinks. You did a great job. I like it. I have some small remarks/ questions Why isn't it listet in PW Modules? Why doesn't it have its own support board in PW Forum. ProcessRedirect is another Module. I think its better to have one support board for each module, even if they are related. I would like to have an option to clear the 'HITS'. EDITJust found it in the directory and even the support board: https://processwire.com/talk/topic/8697-jumplinks/ It isn't listet if I select modules by Modulauthor mikerockett mikerockett or mike-rockett (better to merge) I strongly recommend to unravel Support Board for ProcessJumplinks and ProcessRedirect. Maybe this post could be moved to Jumplinks Support Board too. (Hello Pete?) Hi kixe, Hehe, glad you found it. I would like to have an option to clear the 'HITS'. > I will be adding that option to v1.5. It isn't listet if I select modules by Modulauthor mikerockett mikerockett or mike-rockett (better to merge) > Thanks for picking that up - not sure why that happened. Hoping an admin can pop in and assist with that... Link to comment Share on other sites More sharing options...
adrian Posted November 26, 2015 Share Posted November 26, 2015 Thanks for picking that up - not sure why that happened. Hoping an admin can pop in and assist with that... Hey Mike - looks like you have a lot of aliases in the modules directory. Are all these you? What do you want them all merged to? Link to comment Share on other sites More sharing options...
Mike Rockett Posted November 27, 2015 Author Share Posted November 27, 2015 Hey Mike - looks like you have a lot of aliases in the modules directory. Are all these you? What do you want them all merged to? Screen Shot 2015-11-27 at 5.09.24 AM.png Hey Adrian - yes, please. I guess they should all be merged into the third one, which equates to the named version of my profile display name. Or, even better, they should actually be merged to my GitHub username, which is the last one. That possible? Thanks. (Oh, and I'm not sure about the first one - I may have tried to change my display name to just mike at one point, but didn't think it would affect the modules directory...) Link to comment Share on other sites More sharing options...
adrian Posted November 27, 2015 Share Posted November 27, 2015 Or, even better, they should actually be merged to my GitHub username, which is the last one. That possible? Ok, here you go: http://modules.processwire.com/authors/mikerockett/ I am pretty sure you just have those two modules, correct? I have deleted the account author except for "mike", just in case it wasn't you. Link to comment Share on other sites More sharing options...
Mike Rockett Posted November 27, 2015 Author Share Posted November 27, 2015 Ok, here you go: http://modules.processwire.com/authors/mikerockett/ I am pretty sure you just have those two modules, correct? I have deleted the account author except for "mike", just in case it wasn't you. Great, thanks Adrian. Yeah, just the two modules (for now...) 1 Link to comment Share on other sites More sharing options...
Mike Rockett Posted November 27, 2015 Author Share Posted November 27, 2015 Dev branch bumped to 1.3.5 with an important fix. compileDestinationUrl was not returning the correct data, which resulted in relative redirects. This issue has now been resolved, and the behaviour is the same as the current stable. Looking to push to 1.5 soon for stable release - please test... Thanks! 1 Link to comment Share on other sites More sharing options...
Mike Rockett Posted December 8, 2015 Author Share Posted December 8, 2015 Dev branch updated to 1.3.6 adding support for clearing jumplink hits on an individual basis by means of a new checkbox in the jumplink editor. Note that this only clears the counter. The date of the last hit will always be stored. Everything is looking good at this point. Hoping to push to 1.5 stable in about a week or so - please let me know if you've noticed any issues before I do that. Thanks. Link to comment Share on other sites More sharing options...
cb2004 Posted December 21, 2015 Share Posted December 21, 2015 I am trying to implement Jumplinks on a new site but not sure what is happening, the 404 template is not even being fired. I dont think this is a bug with Jumplinks or ProcessWire to be honest, but wondering if any light could be shed on this. If I go to /example.php, I get a page which just says 'File not found.' and nothing else. If I go to /example then the 404 template is fired. I have setup Jumplinks to try and change /example.php to go to /new-example but it is not kicking in. Server configuration do you think? Link to comment Share on other sites More sharing options...
Wanze Posted December 21, 2015 Share Posted December 21, 2015 How did you add the rule for redirecting example.php, with the leading slash? From the description: Enter a URI that's relative to the root of this installation, without the leading slash. @Mike Rockett Just used this module for the first time, it's great!! Thanks for your work Cheers Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now