Closed (fixed)
Project:
Turn.js Magazines [D7]
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 May 2015 at 16:39 UTC
Updated:
20 Nov 2015 at 10:14 UTC
Jump to comment: Most recent
Comments
Comment #1
mr4711 commentedComment #2
anybodyThats related to your PHP version. Which version do you use?
I'll have to fix it in the module.
Comment #3
mr4711 commentedPHP 5.4.39
Comment #4
anybodyThis is already fixed in the latest .dev-Version.
If the .dev Version works fine for all of you I can create a new stable release.
Comment #5
mr4711 commentedI've done some testing and would say: No the problem is only partly fixed. The warings are not at the "turnjs_magazines" sites any more, but on some of the pages showing a normal content node
It is strange, the pages do not have any link to turnjs_magazines ....
Comment #6
mr4711 commentedComment #7
mr4711 commentedWe do still have tons of this warnings:
Notice: Undefined property: stdClass::$type in turnjs_magazines_entity_view() (line 57 of /homepages/44/d513036722/htdocs/sites/all/modules/turnjs_magazines/turnjs_magazines.module).
Would you help us to get rid of it?
Comment #8
mr4711 commentedWe sill have the problem and are still not experienced in PHP. So please, could someone give us a hint how to get rid of this warning. In the mean time we have many PDFs and tons of warinings in the logfile:
The message is:
Notice: Undefined property: stdClass::$type in turnjs_magazines_entity_view() (line 57 of /homepages/44/d513036722/htdocs/sites/all/modules/turnjs_magazines/turnjs_magazines.module).and the code is:
Is there something we should add in the IF statement?
Comment #9
anybodyPlease try the latest turnjs_magazines 7.x-3.x-dev release if you are using modernizr v3.
Of course in a testing and not in a live environment! ;)
Feedback is welcome if this fixes the problem for you.
Comment #10
mr4711 commentedYep, the missing "isset" was the problem.
Thank you!
Comment #11
anybody