[EP-tech] Re: IRStats2 Beta

Sebastien Francois sf2 at ecs.soton.ac.uk
Wed Jun 26 14:49:19 BST 2013


Hi all,

To those who've been trying IRStats2, I suggest you apply the following 
patch (described in the first comment) from there:

https://github.com/eprints/irstats2/issues/7

Otherwise this might break some other functionalities of your repository 
(such as BatchEdit)... Sorry about that!

Seb.

On 26/06/13 11:01, Jose Martin wrote:
>
> Hi Seb,
>
> That did it! The version I had from irstats2-0.0.3.epm had
>
> if( exists $pins->{"utf-8.head"} && ref( $pins->{"utf-8.head"} ) eq "" )
>
>       {
>
> that prevented the magic from happening.
>
> Thanks,
>
> Jose
>
> *From:*eprints-tech-bounces at ecs.soton.ac.uk 
> [mailto:eprints-tech-bounces at ecs.soton.ac.uk] *On Behalf Of *Sebastien 
> Francois
> *Sent:* 25 June 2013 16:10
> *To:* eprints-tech at ecs.soton.ac.uk
> *Subject:* [EP-tech] Re: IRStats2 Beta
>
> Jose,
>
> Does your trigger looks like the block of code below (I know there 
> were a few different versions hanging around):
>
> $c->add_trigger( EP_TRIGGER_DYNAMIC_TEMPLATE , sub
> {
>     my( %args ) = @_;
>
>     my( $repo, $pins ) = @args{qw/ repository pins/};
>
>     $pins->{"utf-8.head"} = "" if( !exists $pins->{"utf-8.head"} );
>
>     my $protocol = $repo->get_secure ? 'https':'http';
>
>     $pins->{"utf-8.head"} .= <<GCHARTS;
>
> <!-- IRStats2 -->
> <script type="text/javascript" 
> src="$protocol://www.google.com/jsapi">// <!-- No script --></script>
> <script type="text/javascript">google.load("visualization", "1", 
> {packages:["corechart", "geochart"]});</script>
> <!-- end IRStats2 -->
> GCHARTS
>
>     return EP_TRIGGER_OK;
> } );
>
>
> If not, could you replace with the code above & re-try please?
>
> Thanks again!
> Seb
>
>
> On 25/06/13 15:47, Jose Martin wrote:
>
>     Hi Seb,
>
>       
>
>     1- Yes, I did restart Apache
>
>       
>
>     2- The <epc:pin ref="head"/> is in place. I am using a template inside a theme, but that should not interfere, I guess.
>
>       
>
>     3- Yes, it is, but when loading the cgi/stats/report page a Dumper($pins) outputs:
>
>       
>
>        $VAR1 = {
>
>                'head' => bless( do{\(my $o = '47743427226608')}, 'XML::LibXML::DocumentFragment' ),
>
>                'page' => bless( do{\(my $o = '47743215127744')}, 'XML::LibXML::DocumentFragment' ),
>
>                'languages' => bless( do{\(my $o = '47743427226496')}, 'XML::LibXML::DocumentFragment' ),
>
>                'title' => bless( do{\(my $o = '47743215188816')}, 'XML::LibXML::DocumentFragment' ),
>
>                'login_status' => bless( do{\(my $o = '47743427161872')}, 'XML::LibXML::Element' )
>
>              };
>
>       
>
>         so it seems to never get into the if block that writes the IRStats2 scripts.
>
>       
>
>     Any other useful debugging that might help?
>
>       
>
>     My pleasure testing IRStats 2...   It looks great!
>
>       
>
>     Thanks,
>
>       
>
>        Jose
>
>       
>
>       
>
>     -----Original Message-----
>
>     From:eprints-tech-bounces at ecs.soton.ac.uk  <mailto:eprints-tech-bounces at ecs.soton.ac.uk>  [mailto:eprints-tech-bounces at ecs.soton.ac.uk] On Behalf Of Sebastien Francois
>
>     Sent: 25 June 2013 15:33
>
>     To:eprints-tech at ecs.soton.ac.uk  <mailto:eprints-tech at ecs.soton.ac.uk>
>
>     Subject: [EP-tech] Re: IRStats2 Beta
>
>       
>
>     Hi Jose,
>
>       
>
>     A few pointers:
>
>       
>
>     1- did you restart Apache? this is necessary for the trigger to be loaded
>
>       
>
>     2- in your template file(s) (usually under archives/<id>/cfg/lang/en/templates), do you have the following line:
>
>       
>
>     <epc:pin ref="head"/>
>
>       
>
>     in your <head> section?
>
>       
>
>     That "pin" was added in 3.3 but may be missing on some repositories upgraded from older EPrints version.
>
>       
>
>     3- is the trigger called by EPrints? You could for instance add a "print STDERR" debug message inside the trigger to see if it's properly called.
>
>       
>
>       
>
>     Thanks to everyone for testing IRStats2, I will be updating the
>
>     code/writing a FAQ soon to tackle those issues.
>
>       
>
>     Kind regards,
>
>     Seb.
>
>       
>
>       
>
>     On 25/06/13 15:14, Jose Martin wrote:
>
>         Hi Seb,
>
>           
>
>         I have installed IRStats2 on a 3.3.10 and it does not show the Charts (keeps displaying the "Loading..." image).
>
>           
>
>         I used the irstats2-0.0.3.epm package and upgraded Prototype to 1.7.1 applyinghttps://github.com/eprints/eprints/commit/f7a9088aae29b8732cb5d8edd83d17e94f069e63
>
>           
>
>         The browser reports no network or console errors, although I have noticed that it does not request "jsapi" from Google as doeshttp://files.eprints.org/cgi/stats/report. This looks like should be triggered in z_irstats2.pl, but does not seem to happen. Any ideas?
>
>           
>
>         BTW, irstats2-0.0.3.epm did not include the images/irstats2/datepicker folder.
>
>           
>
>         Thanks,
>
>           
>
>             Jose
>
>           
>
>           
>
>         -----Original Message-----
>
>         From:eprints-tech-bounces at ecs.soton.ac.uk  <mailto:eprints-tech-bounces at ecs.soton.ac.uk>  [mailto:eprints-tech-bounces at ecs.soton.ac.uk] On Behalf Of Sebastien Francois
>
>         Sent: 30 May 2013 16:18
>
>         To:eprints-tech at ecs.soton.ac.uk  <mailto:eprints-tech at ecs.soton.ac.uk>
>
>         Subject: [EP-tech] IRStats2 Beta
>
>           
>
>         Hi all,
>
>           
>
>         I have added IRStats2 to the eprints GitHub so you may now download and use/test it.
>
>           
>
>         EPrints 3.3 users may use the bazaar package available from there:
>
>         https://github.com/eprints/irstats2/blob/master/epm/irstats2-0.0.2.epm
>
>         (note that it's not yet live on the Bazaar-store). Even so you can install the package easily this way, you still need to run a script to process the stats. That is not yet automated.
>
>           
>
>         EPrints 3.2 users will have to copy files by hand, it's pretty straight-forward and the INSTALL file contains some instructions.
>
>           
>
>         It's not quite the final/stable release yet but it has been thoroughly tested on a few "real" repositories and it's unlikely to break a repository anyway.
>
>           
>
>         I hope you'll enjoy, please contribute (ideas / comments / bugs etc) directly to me or to GitHub.
>
>           
>
>         Seb.
>
>           
>
>         PS: yes, it can run in parallel to IRStats1.
>
>         *** Options:http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
>
>         *** Archive:http://www.eprints.org/tech.php/
>
>         *** EPrints community wiki:http://wiki.eprints.org/
>
>           
>
>         *** Options:http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
>
>         *** Archive:http://www.eprints.org/tech.php/
>
>         *** EPrints community wiki:http://wiki.eprints.org/
>
>       
>
>     *** Options:http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
>
>     *** Archive:http://www.eprints.org/tech.php/
>
>     *** EPrints community wiki:http://wiki.eprints.org/
>
>       
>
>     *** Options:http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
>
>     *** Archive:http://www.eprints.org/tech.php/
>
>     *** EPrints community wiki:http://wiki.eprints.org/
>
>
>
> *** Options: http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
> *** Archive: http://www.eprints.org/tech.php/
> *** EPrints community wiki: http://wiki.eprints.org/

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mailman.ecs.soton.ac.uk/pipermail/eprints-tech/attachments/20130626/9d196812/attachment.html 


More information about the Eprints-tech mailing list