EPrints Technical Mailing List Archive
Message: #07160
< Previous (by date) | Next (by date) > | < Previous (in thread) | Next (in thread) > | Messages - Most Recent First | Threads - Most Recent First
Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- To: "eprints-tech@ecs.soton.ac.uk" <eprints-tech@ecs.soton.ac.uk>
- Subject: Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: Liam Green-Hughes <L.E.Green-Hughes@kent.ac.uk>
- Date: Fri, 23 Feb 2018 11:39:33 +0000
Hi Monica,
I've had the same problem with statistics boxes disappearing. Looks to be the same issue as you had as when I follow what you did here the same symptoms occur. I was just wondering how it went with removing the return
0 if $self->{session}->get_secure; line. I have no idea why it is there! Were there any side effects?
Thanks Liam
Liam Green-Hughes Library Systems Developer
University of Kent
From: eprints-tech-bounces@ecs.soton.ac.uk <eprints-tech-bounces@ecs.soton.ac.uk> on behalf of Monica Wood <monica.wood@utas.edu.au>
Sent: 09 January 2018 01:29 To: eprints-tech@ecs.soton.ac.uk Subject: Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page Update on this issue.
I compared the index.html in /html/en/archive/…. Of a single record, both with one that was generated using bin/generate_abstracts and one using bin/epadmin refresh_abstracts.
I noticed the one using generate_abstracts defined the http_root and http_cgiroot as http, while the refresh_abstracts was https.
So I figured that generate_abstracts was run locally, so it wasn’t forced through https, while refresh_abstracts was.
It had to be something that the Screen::EPrint::Box didn’t like about being generated through https.
Well the issue isn’t with Screen::Eprint::Box per say, it was the implementations of Irstats and Altmetrics using this plugin (lib/plugins/EPrints/Plugin/Screen/Eprint/Box/[Altmetric.pm|Irstats.pm]).
Both in their can_be_viewed subs have the line: return 0 if $self->{session}->get_secure;
So it just simply won’t show the box on a page generated through https.
I have commented this line out and it’s now showing on pages generated through refresh_abstracts.
I guess my question now is, what is the importance of this line and what am I breaking by commenting it out? :/
Thanks everyone Monica
University of Tasmania
From: <eprints-tech-bounces@ecs.soton.ac.uk> on behalf of John Salter <J.Salter@leeds.ac.uk>
Maybe try this: Find an item without the boxes. Save the html source of the page Run generate_abstracts to make the boxes reappear. Save the html source of this.
Diff the two html sources - what's the difference? That might point us toward what's going on…
From: eprints-tech-bounces@ecs.soton.ac.uk [mailto:eprints-tech-bounces@ecs.soton.ac.uk]
On Behalf Of Monica Wood
As it renders fine when i run generate_abstracts, but then disappears when you edit a record or refresh abstracts, i wouldn't imagine its the _javascript_. I haven't changed the templates either. It use to serve all pages (except pages like login) through both http and https, but any requests to http is now redirected to https. If you look at our repository you will probably see newly created items with no boxes and older records have the boxes. If i generate_abstracts on the new ones then the boxes will appear. I just dont understand what could possibly cause this. Monica
On Thu, Jan 4, 2018 at 8:19 PM +1100, "John Salter" <J.Salter@leeds.ac.uk> wrote:
|
- Follow-Ups:
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: Liam Green-Hughes <L.E.Green-Hughes@kent.ac.uk>
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- References:
- [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: Monica Wood <monica.wood@utas.edu.au>
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: John Salter <J.Salter@leeds.ac.uk>
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: Monica Wood <monica.wood@utas.edu.au>
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: John Salter <J.Salter@leeds.ac.uk>
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: Monica Wood <monica.wood@utas.edu.au>
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: John Salter <J.Salter@leeds.ac.uk>
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: Monica Wood <monica.wood@utas.edu.au>
- Re: [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- From: Liam Green-Hughes <L.E.Green-Hughes@kent.ac.uk>
- [EP-tech] Refresh Abstracts removing Boxes from Summary Page
- Prev by Date: [EP-tech] Change Admin View Configuration Path
- Next by Date: Re: [EP-tech] Change Admin View Configuration Path
- Previous by thread: [EP-tech] Sort view with creators_name and corp_creators
- Index(es):