Gossamer Forum: Products: Gossamer Links: Development, Plugins and Globals: [ NEW PLUGIN ] PeopleWhoClicked v1

Veteran / Moderator (18441 posts)

Jan 18, 2008, 2:47 AM

Post #1 of 49

Shortcut

[ NEW PLUGIN ] PeopleWhoClicked v1

Hi,

I'm pleased to announce the release of our latest plugin, PeopleWhoClicked. The details are as follows:

NB: This plugin is part of both the "ULTRA" and "ULTRA PRO" Packages

Price: $50 [ details ]

Plugin Description

This plugin records the activity's of a user, in terms of which links they last accessed, and which ones they then went onto access. This is a great tool, as generally people goto links that are related - so you can provide a nice simple interface, for them to click on related links. It works based on how many occurrences the relation has had (i.e if there are 3 occurrences of someone going from linkID 123 to linkID 234, then that would show up higher on the list)

It works in a similar way to Amazon's "people who clicked this link, also visited..." section.

Plugin Requirements

1. Gossamer Links 3.0.0+

Features

Shows highest ranking "related" links towards the top of the related loop, and then working downwards.
Great tool for showing to your users.
Easy to setup.
Works with both detailed pages, and also jump.cgi.

Once a few people have it setup on their site, I will post a demo. The couple of sites its being used on already, don't want to make their site listed as a demo (which is fair enough), but if you install this plugin - and don't mind sharing it as an example site, then please PM or email me.

As always, if you have any questions - please don't hesitate to ask (here, or via email/PM)

Cheers

Andy (mod)
andy@ultranerds.co.uk

Last edited by:

Andy: Jan 18, 2008, 2:51 AM

Jan 18, 2008, 4:26 AM

Enthusiast (893 posts)

Jan 18, 2008, 4:26 AM

Post #2 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi Andy,
I'm using detailed pages
www.domain/Detailed/123.html
and inside my detailed page I have the download link with jump.cgi.

So I'm using detailed pages AND jump.cgi.
What part of the code in the readme file do I need?
Do I need both or only one of these parts?

Thanks
Matthias

Matthias
gpaed.de

Jan 18, 2008, 4:30 AM

Veteran / Moderator (18441 posts)

Jan 18, 2008, 4:30 AM

Post #3 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

You will be ok with the detailed code, and jump.cgi.

i.e just add the following to detailed.html:

Code:
<%if Username%> 
  <%Plugins::PeopleWhoClicked::Track_Detailed($ID)%> 
<%endif%>

..and then you can use this to show the related link (in detailed.html , or even link.html if you want):

Code:
  <%Plugins::PeopleWhoClicked::Get_Related($ID)%> 

  <%if related_links_loop.length%> 
      <h2>People also visited...</h2> 
       <%loop related_links_loop%> 
          <%include link.html%> 
       <%endloop%> 
  <%endif%>

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 18, 2008, 4:51 AM

Enthusiast (893 posts)

Jan 18, 2008, 4:51 AM

Post #4 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi Andy,
o.k. installtion is done.

Another thing.
The plugin shows on detailed page, the same link of the detailed page in the "people who clicked"-part?
But I it would be better if the link is not not shown twice.

Matthias

Matthias
gpaed.de

Jan 18, 2008, 4:54 AM

Veteran / Moderator (18441 posts)

Jan 18, 2008, 4:54 AM

Post #5 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Quote:

Another thing.
The plugin shows on detailed page, the same link of the detailed page in the "people who clicked"-part?
But I it would be better if the link is not not shown twice.

I'm not sure what you mean? Can you PM/email me an example?

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 18, 2008, 5:11 AM

Veteran / Moderator (18441 posts)

Jan 18, 2008, 5:11 AM

Post #6 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

I see the problem :)

Please try the latest version from our Members Area.

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 18, 2008, 5:23 AM

Enthusiast (893 posts)

Jan 18, 2008, 5:23 AM

Post #7 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi Andy,
now it's working fine. I do some more testing later.
Thanks
Matthias

Matthias
gpaed.de

Jan 18, 2008, 5:26 AM

Veteran / Moderator (18441 posts)

Jan 18, 2008, 5:26 AM

Post #8 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Glad to hear it :)

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 18, 2008, 4:20 PM

Clint

User (387 posts)

Jan 18, 2008, 4:20 PM

Post #9 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi Andy:

Hmmm .... this sounds like a very worthwhile/useful plugin ...

Does this also work with detailed.cgi (written sometime ago by Pugdog) .....

Thanks,

Clint.
--------------------------
http://AffiliatesDirectory.com
The Affiliate Programs Directory

Jan 19, 2008, 4:42 AM

User (497 posts)

Jan 19, 2008, 4:42 AM

Post #10 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Cool! Will try it. Just to earn 50 bucks Wink

UnReal Network

Jan 19, 2008, 4:58 AM

Veteran / Moderator (18441 posts)

Jan 19, 2008, 4:58 AM

Post #11 of 49

Shortcut

Re: [Clint] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Clint wrote:

Hmmm .... this sounds like a very worthwhile/useful plugin ...

Does this also work with detailed.cgi (written sometime ago by Pugdog) .....

Sure does :)

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 19, 2008, 10:46 PM

socrates

Enthusiast (832 posts)

Jan 19, 2008, 10:46 PM

Post #12 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Nice and works in detail_page.cgi but ALL links under people who clicked show new, updated and popular tags next to them. How can it be made so they also show only if a specific link is new or updated (or just show no tags at all).

Also, it would be nice to show/limit only 3-5 links under people who clicked or the page will get too long.

Thanks

Last edited by:

socrates: Jan 19, 2008, 10:50 PM

Jan 20, 2008, 3:55 AM

Veteran / Moderator (18441 posts)

Jan 20, 2008, 3:55 AM

Post #13 of 49

Shortcut

Re: [socrates] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

I've just uploaded version 1.2.

Quote:

I've fixed this now :) Its because the database holds "Yes" or "No", but the template only does <%if isNew%>...<%endif%>, which is comes up as "true" if ANY text is passed in (unless its a 0 or 1). Anyway, its fixed now Cool

Quote:

Also, it would be nice to show/limit only 3-5 links under people who clicked or the page will get too long.

Try:

<%Plugins::PeopleWhoClicked::Get_Related($ID,5)%>

The 2nd option is how many links you want to shown :)

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 20, 2008, 10:49 AM

User (339 posts)

Jan 20, 2008, 10:49 AM

Post #14 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Andy,

I just bought the plugin and from readme I see it works only when user logged in, what happens with visits from other users, who are not logged in?

Cheers,
Boris

Facebook, Twitter and Google+ Auth for GLinks and GCommunity | reCAPTCHA for GLinks | Free GLinks Plugins

Jan 20, 2008, 11:06 AM

Enthusiast (893 posts)

Jan 20, 2008, 11:06 AM

Post #15 of 49

Shortcut

Re: [eupos] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi Boris,
I think when use

Code:
<%Plugins::PeopleWhoClicked::Track_Detailed($ID)%>

instead of

Code:
<%if Username%>  
  <%Plugins::PeopleWhoClicked::Track_Detailed($ID)%>  
<%endif%>

the hits on detailed pages are counted for guests and registered users...

Matthias

Matthias
gpaed.de

Jan 20, 2008, 11:18 AM

Veteran / Moderator (18441 posts)

Jan 20, 2008, 11:18 AM

Post #16 of 49

Shortcut

Re: [eupos] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi Boris,

Yes, Matthias isn't quite right :)

It requires a user to be logged in, so it can track who's looking at what link, and what link they look at next :)

So, basically - the plugin will work fine without being logged in - but to track the page's they go to, it needs a username.

I may make it so it works on IP addresses in the future - as well as usernames , as this will avoid the need to be logged in :)

Hope that helps.

Cheers

Andy (mod)
andy@ultranerds.co.uk

Last edited by:

Andy: Jan 20, 2008, 11:19 AM

Jan 20, 2008, 11:19 AM

User (339 posts)

Jan 20, 2008, 11:19 AM

Post #17 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Nope :(

I've looked in the code and if there is no Username it will not count the hit.

Cheers,
Boris

Facebook, Twitter and Google+ Auth for GLinks and GCommunity | reCAPTCHA for GLinks | Free GLinks Plugins

Jan 20, 2008, 11:24 AM

User (339 posts)

Jan 20, 2008, 11:24 AM

Post #18 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Ah, you just posted before me :)

When you plan to change it? As because of this, it is useless for me at this moment, most of my users are not log in until they need to use some feature that needs a username.

Cheers,
Boris

Facebook, Twitter and Google+ Auth for GLinks and GCommunity | reCAPTCHA for GLinks | Free GLinks Plugins

Jan 20, 2008, 11:36 AM

Veteran / Moderator (18441 posts)

Jan 20, 2008, 11:36 AM

Post #19 of 49

Shortcut

Re: [eupos] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

I'll try the change in a few minutes =)

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 20, 2008, 11:53 AM

User (339 posts)

Jan 20, 2008, 11:53 AM

Post #20 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Thank you

Cheers,
Boris

Facebook, Twitter and Google+ Auth for GLinks and GCommunity | reCAPTCHA for GLinks | Free GLinks Plugins

Jan 20, 2008, 12:25 PM

Veteran / Moderator (18441 posts)

Jan 20, 2008, 12:25 PM

Post #21 of 49

Shortcut

Re: [eupos] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Please give version 1.3 a go (from our Members Area)

It now works by using the "Username", or if that doesn't exist - it uses REMOTE_ADDR (their IP address).

If both don't exist (i.e they are hiding their IP for whatever reason), it won't bother tracking them - as it would be totally inaccurate data :)

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 20, 2008, 3:20 PM

Enthusiast (893 posts)

Jan 20, 2008, 3:20 PM

Post #22 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Matthias70 wrote:

The plugin shows on detailed page, the same link of the detailed page in the "people who clicked"-part?
But I it would be better if the link is not not shown twice.

Matthias

Hi Andy,
it happens again in version 1.3 !

Matthias

Matthias
gpaed.de

Jan 20, 2008, 10:51 PM

Veteran / Moderator (18441 posts)

Jan 20, 2008, 10:51 PM

Post #23 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Mmm.. really? The code that stopped it in version 1.3 hasn't been removed Unsure

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 20, 2008, 11:08 PM

Veteran / Moderator (18441 posts)

Jan 20, 2008, 11:08 PM

Post #24 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

I've just double checked with detailed pages and jump.cgi (using at the same time, with 2 different links), and can't get this problems to occur :/

I've added some debug information into the new version available from our site (still version 1.3).

In PeopleWhoClicked.pm, find:

Code:
my $debug = 0;

..and set to:

Code:
my $debug = 1;

Then, try viewing links.

If an entry was added to the glinks_PeopleWhoClickedRelations table, it will show "ADDED!" - if not, then it was skipped

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 21, 2008, 4:17 AM

Enthusiast (893 posts)

Jan 21, 2008, 4:17 AM

Post #25 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Upps, sorry I still was using 1.2 Angelic

Thanks
Matthias

Matthias
gpaed.de

Jan 21, 2008, 4:23 AM

Veteran / Moderator (18441 posts)

Jan 21, 2008, 4:23 AM

Post #26 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

No problem :D

BTW to delete the links that are referencing themseleves, just use:

Code:
DELETE FROM glinks_PeopleWhoClickedRelations WHERE LinkID = RelatedID

Then, to check (lets say tomorrow, for example) if this is still a problem - run:

Code:
SELECT * FROM glinks_PeopleWhoClickedRelations WHERE LinkID = RelatedID

If alls working well, it should give "no results" - but if its still cross-referencing itself, you will get some rows (we don't want this though Angelic

)

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 21, 2008, 9:16 PM

socrates

Enthusiast (832 posts)

Jan 21, 2008, 9:16 PM

Post #27 of 49

Shortcut

Re: [Matthias70] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

I am not sure if there is another plugin or global to recently clicked/visited links and if not is it possible to use this one.

I would like to show a list of recently visited links (for logged in users) that they have already clicked on, either on account page or on a different page?

Thanks

Jan 22, 2008, 12:51 AM

Veteran / Moderator (18441 posts)

Jan 22, 2008, 12:51 AM

Post #28 of 49

Shortcut

Re: [socrates] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Mmm... I guess this could be done with this plugin.

I guess this could be done with the ClickTrack table. I'll see what I can come up with.

Cheers

Andy (mod)
andy@ultranerds.co.uk

Feb 23, 2008, 2:25 PM

User (497 posts)

Feb 23, 2008, 2:25 PM

Post #29 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

There is error when use click on "Random Link"

Code:
A fatal error has occured: 

LinkID cannot contain the value 'random' at /admin/Plugins/PeopleWhoClicked.pm line 64. 

Please enable debugging in setup for more details.

I also found this in my error log but don't know how it is produced.

Code:
Column LinkID cannot be left blank at /admin/Plugins/PeopleWhoClicked.pm line 64.

Regards.

UnReal Network

Last edited by:

deadroot: Feb 23, 2008, 2:26 PM

Feb 24, 2008, 2:24 AM

Veteran / Moderator (18441 posts)

Feb 24, 2008, 2:24 AM

Post #30 of 49

Shortcut

Re: [deadroot] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Thanks for the heads up. I've fixed this up now in 1.4. You can get it from the Members Area.

Cheers

Andy (mod)
andy@ultranerds.co.uk

Feb 24, 2008, 2:29 AM

User (497 posts)

Feb 24, 2008, 2:29 AM

Post #31 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Still got error.

Code:
A fatal error has occured: 

LinkID cannot contain the value 'random' /admin/Plugins/PeopleWhoClicked.pm line 80. 

Please enable debugging in setup for more details.

UnReal Network

Feb 24, 2008, 3:16 AM

Veteran / Moderator (18441 posts)

Feb 24, 2008, 3:16 AM

Post #32 of 49

Shortcut

Re: [deadroot] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Whoops, sorry - should be fixed now. There was another bit that adds a link ID to the database =)

Cheers

Andy (mod)
andy@ultranerds.co.uk

May 7, 2008, 9:56 AM

Gorospe

User (175 posts)

May 7, 2008, 9:56 AM

Post #33 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Love the plugin.
Thanks Andy.

Ok, so the display is say 5 links, all going down.
I wish to display 5 links but going sideways.

Instead of showing all info from link.html, I only want to show the small image asociated with it. Ive got the image part done, no prob. But I want them to display side by side and not vertically.

I cant get the loop table right.

Anyone can help?

Thanks,

Juan Carlos

May 7, 2008, 10:02 AM

Veteran / Moderator (18441 posts)

May 7, 2008, 10:02 AM

Post #34 of 49

Shortcut

Re: [Gorospe] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Does this work?

Code:
  <%Plugins::PeopleWhoClicked::Get_Related($ID)%> 

  <%if related_links_loop.length%> 
      <h2>People also visited...</h2> 
       <%loop related_links_loop%> 
          <%-- show the image here --%> 
       <%endloop%> 
  <%endif%>

You may need to add some "padding" to the images (so they don't show next to each other).

Something like this:

Code:
style="padding: 5px 5px 5px 5px;"

Cheers

Andy (mod)
andy@ultranerds.co.uk

May 7, 2008, 10:17 AM

Gorospe

User (175 posts)

May 7, 2008, 10:17 AM

Post #35 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi, thanks for the quick reply.

Nop, it doesnt work. Same result.

In link.html ive added an if the page id is detailed shot only the image and title.

Hmm I thought the code needed would be similar to the categories code. Where a table is actualy drawn.

Any ideas?

May 7, 2008, 10:27 AM

Veteran / Moderator (18441 posts)

May 7, 2008, 10:27 AM

Post #36 of 49

Shortcut

Re: [Gorospe] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Can you PM me, with the template your doing it in? I'll have a look (that should work fine)

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 5, 2009, 8:17 AM

User (497 posts)

Jan 5, 2009, 8:17 AM

Post #37 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Today I found lot of this lines in my error log:

Code:
LinkID cannot contain the value '4662//modules/My_eGallery/public/displayCategory.php?basepath=http://www.wehrmachtsgespanne.com/apboard/bajo-id.txt???' at /home/myserver/Plugins/PeopleWhoClicked.pm line 81.

Now, what is this? Does this has something to do with people try to hack into? I don't know anything about this site www.wehrmachtsgespanne.com and that site isn't in my directory.

Regards.

UnReal Network

Jan 5, 2009, 9:20 AM

Veteran / Moderator (18441 posts)

Jan 5, 2009, 9:20 AM

Post #38 of 49

Shortcut

Re: [DeadMan] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Mmm.. looks like it may be someone trying to find a hole in your system - with something like:

page.cgi?g=Detailed/1234http://www.domain.com

A way around this, would be in PeopleWhoClicked.pm, find:

Code:
sub handle_jump { 
# ----------------------------------------------------------------------------- 
# This subroutine will be called whenever the hook 'handle_jump' is run.  You 
# should call $PLG->action(STOP) if you don't want the regular 
# 'handle_jump' code to run, otherwise the code will continue as normal. 
# 
    my (@args) = @_;

..and add this after: my (@args) = @_;

Code:
if ($IN->param('ID') !~ /^\d+$/) { 
  print $IN->header; 
  print qq|INVAILD ID!!!|; 
 exit; 
}

Hopefully that'll stop it.

Cheers

Andy (mod)
andy@ultranerds.co.uk

Jan 13, 2009, 4:56 AM

nir

User (396 posts)

Jan 13, 2009, 4:56 AM

Post #39 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi Andy, can you insert the option to limit the lists of the People Who Clicked just to a category and all here subcategory
<%Plugins::PeopleWhoClicked::Track_Detailed($ID,3,catid)%>

Jan 13, 2009, 5:03 AM

Veteran / Moderator (18441 posts)

Jan 13, 2009, 5:03 AM

Post #40 of 49

Shortcut

Re: [nir] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Mmm.. not sure off hand. Been a while since I did much with that plugin. Kinda bogged down at the moment, so would have to take a look at some point in the future.

Cheers

Andy (mod)
andy@ultranerds.co.uk

Apr 25, 2011, 1:50 AM

User (497 posts)

Apr 25, 2011, 1:50 AM

Post #41 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

I have same problem here like in NewInCategory plugin.

Links are display like http://www.domain.com/category/Link.php although I'm using this type of URL http://www.domain.com/Detailed/Category/Link.php

Missing "Detailed" in URL when displaying links in page where I put "People who clicked also visit..:"

Regards.

UnReal Network

Apr 25, 2011, 2:40 AM

Veteran / Moderator (18441 posts)

Apr 25, 2011, 2:40 AM

Post #42 of 49

Shortcut

Re: [DeadMan] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

I'll have to go through and fix up those plugins when I get a chance =) Its cos the way "detailed" pages are set, was changed in later versions. You can fix this in /Plugins/PeopleWhoClicked.pm, by changing:

Code:
if ($CFG->{build_detailed}) { $link->{detailed_url} = $CFG->{build_root_url} . "/" . $DB->table('Links')->detailed_url( $link->{ID} ); }

..to:

Code:
if ($CFG->{build_detailed}) { $link->{detailed_url} = $CFG->{build_detail_url} . "/" . $DB->table('Links')->detailed_url( $link->{ID} ); }

Cheers

Andy (mod)
andy@ultranerds.co.uk

Apr 25, 2011, 2:46 AM

User (497 posts)

Apr 25, 2011, 2:46 AM

Post #43 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Done. Working. Thanks.

Regards.

UnReal Network

May 4, 2011, 1:19 PM

User (497 posts)

May 4, 2011, 1:19 PM

Post #44 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Andy wrote:

Code:
sub handle_jump { 
# ----------------------------------------------------------------------------- 
# This subroutine will be called whenever the hook 'handle_jump' is run.  You 
# should call $PLG->action(STOP) if you don't want the regular 
# 'handle_jump' code to run, otherwise the code will continue as normal. 
# 
    my (@args) = @_;

..and add this after: my (@args) = @_;

Code:
if ($IN->param('ID') !~ /^\d+$/) { 
  print $IN->header; 
  print qq|INVAILD ID!!!|; 
 exit; 
}

Hopefully that'll stop it.

Cheers

Is this reason I'm getting INVAILD ID!!! message for my random link url which is like http://www.domain.com/jump.cgi?ID=random;Detailed=1 ?

Regards.

UnReal Network

May 5, 2011, 12:08 AM

Veteran / Moderator (18441 posts)

May 5, 2011, 12:08 AM

Post #45 of 49

Shortcut

Re: [DeadMan] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Hi,

Yeah, it looks it. What you wanna probably do, is remove:

Code:
if ($IN->param('ID') !~ /^\d+$/) { 
     print $IN->header; 
     print qq|INVAILD ID!!!|; 
     exit; 
}

..and change:

Code:
if ($linkid !~ /\d+/) { return @args; }

..to:

Code:
if ($linkid !~ /^\d+$/) { return @args; }

Hope that helps.

Cheers

Andy (mod)
andy@ultranerds.co.uk

May 5, 2011, 11:00 AM

User (497 posts)

May 5, 2011, 11:00 AM

Post #46 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Well, it did fix problem with random link. Hope plugin is working as expected :)

Regards.

UnReal Network

May 5, 2011, 11:06 AM

Veteran / Moderator (18441 posts)

May 5, 2011, 11:06 AM

Post #47 of 49

Shortcut

Re: [DeadMan] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Yeah, it should do =)

The problem with the old regex:

Code:
/\d+/

..is that it would also mathc:

1234Foo

Foo3343

etc

With the new regex:

Code:
/^\d+$/

..it will ONLY match numbers, i.e:

123
5345
343445
etc

I'll have to get the plugin updated at some point Smile

Cheers

Andy (mod)
andy@ultranerds.co.uk

Feb 5, 2014, 12:28 PM

User (497 posts)

Feb 5, 2014, 12:28 PM

Post #48 of 49

Shortcut

Re: [Andy] [ NEW PLUGIN ] PeopleWhoClicked v1 In reply to

Just interested, if I delete some link from database, will data about that link also delete from PeopleWhoClicked data?

Regards.

UnReal Network

Feb 5, 2014, 11:45 PM