Closed (fixed)
Project:
Page Title
Version:
7.x-2.5
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Jul 2011 at 09:48 UTC
Updated:
22 May 2012 at 10:40 UTC
Jump to comment: Most recent file
Hello
I tried to use page_title in views (without any success for now) so I made a display in a view with the page_title option. Because I failed to make it work, I've deleted this display, and left my view has it was before my test.
Since then, when I go on pages made by views, I got this srtict warning :
Strict warning : Only variables should be passed by reference dans views_page_title_pattern_alter()(line 33 in /sites/all/modules/page_title/modules/views.page_title.inc).
Anyone has an idea from where it can come from ?
Maybe must I delete something in the DB ?
Thanks for your help.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | page_title-array_shift-1206164-4.patch | 682 bytes | peximo |
| #10 | page_title-array_shift-1206164-4.patch | 0 bytes | peximo |
Comments
Comment #1
nicholasthompsonI wonder what's being passed into that function which isn't a variable?!
Comment #2
anouIs that suppose to mean that I can't pass some fixed string, must only use the tokens ?
...
Comment #3
nicholasthompsonYou cant alter a fixed string - however you can alter a string assigned to a variable... eg:
Bad:
Good
Comment #4
Marshall_Kennard commentedTo fix the original problem in views.page_title.inc, replace line 33 with this:
// $h = array_shift(array_splice($h, count($args)-1, 1));
$hh = array_splice($h, count($args)-1, 1);
$h = array_shift($hh);
Comment #5
awm commentedsubscribe
Comment #6
wusel commentedI get the following error-message:
Please help me.
Thanks in advance
Comment #7
Lostmonkey commentedI get the exact same error-message (#6), did you solve it?
Comment #8
a.milkovskyMarshall_Kennard , thanks
Comment #9
Lios commentedThat is the same message as the original post and the solution is in post #4...
Comment #10
peximo commentedAttached patch from comment #4
Comment #11
peximo commentedSorry empty patch; this should be right.
Comment #12
plachLooks good and works as advertised.
Comment #13
IckZ commentedworks! thanks for sharing :)
Comment #14
dercheffeAn update of the module which includes this patch would be great :)
Comment #15
nicholasthompsonThis is fixed and committed into Dev branches for D6 and D7. Tagged release soon!