Re: [VuFind-General] [SPAM] Re: [SPAM] Similar items only for e-books

[Demian K. <dem...@vi...>]

It looks like you have Guardian reviews turned on?  Is this intentional?  If not, turn them off and it should go away.  If you do want them, try changing line 443 to:

            if ($data && isset($data['response']['results'])) {

That should get rid of the first notice.

Similar checks may be possible to eliminate the patron-related notice.

In the snippet you show below, there are spaces after the dollar sign in "$ review".  Is this an email-related corruption, or does the code really look like that?  If the code really looks like that, it may be part of the problem too.

Once you've solved some of these problems, feel free to send revised error dumps -- hopefully that will help focus in on the problem.

- Demian

> -----Original Message-----
> From: Steffi_136 [mailto:ste...@gm...]
> Sent: Thursday, August 22, 2013 3:50 PM
> To: vuf...@li...
> Subject: [VuFind-General] [SPAM] Re: [SPAM] Similar items only for e-books
> 
> These are the error messages that are produced when my programmer called the
> page http://www.katzenbibliothek.de/Record/1615:
> 
> [Sat Aug 17 15:58:00 2013] [error] [client 91.64.60.180] PHP Notice:
> Undefined index: results in /usr/local/vufind/web/sys/Reviews.php on line
> 446, referer: http://katzenbibliothek.de/
> [Sat Aug 17 15:58:00 2013] [error] [client 91.64.60.180] PHP Warning:
> Invalid argument supplied for foreach() in
> /usr/local/vufind/web/sys/Reviews.php on line 446, referer:
> http://katzenbibliothek.de/
> [Sat Aug 17 15:58:00 2013] [error] [client 91.64.60.180] PHP Notice:
> Undefined variable: patron in
> /usr/local/vufind/web/services/Record/Record.php on line 241, referer:
> http://katzenbibliothek.de/
> 
> To the /usr/local/vufind/web/sys/Reviews.php file no changes were made. So
> this is the original file from VuFind. The undefined variable 'patron' is
> probably a result of errors, that results of a lack of array processing,
> since the foreach loop gets passed an array.
> 
> The point at which the HTML is blown off in view-reviews.tpl is the
> following:
> 
> <p class="summary"> {$ review.Content}
> {if! $ review.Content && $ review.ReviewURL  } {translate text = "Read the
> full review online ..."} <{$review.ReviewURL}> </ a> {/ if}
> </ p>
> 
> It seems to be that the return value from Amazon annuls the following HTML
> code in the form of an IFrame string.
> 
> Here is the text file containing the error logs and the complete source code
> extract from the template.
> errorlog.txt
> <http://vufind.2307425.n4.nabble.com/file/n4663465/errorlog.txt>
> 
> We hope this can help to find the fault.
> 
> Thanks a lot Steffi.
> 
> 
> 
> -----
> Stephanie Funk
> M.A. Library and Information Science
> http://www.katzenbibliothek.de
> --
> View this message in context: http://vufind.2307425.n4.nabble.com/Similar-
> items-only-for-e-books-tp4663345p4663465.html
> Sent from the vufind-general mailing list archive at Nabble.com.
> 
> ------------------------------------------------------------------------------
> Introducing Performance Central, a new site from SourceForge and
> AppDynamics. Performance Central is your source for news, insights,
> analysis and resources for efficient Application Performance Management.
> Visit us today!
> http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.clktrk
> _______________________________________________
> VuFind-General mailing list
> VuF...@li...
> https://lists.sourceforge.net/lists/listinfo/vufind-general