Closed (outdated)
Project:
Search API
Version:
7.x-1.x-dev
Component:
Views integration
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
1 Oct 2013 at 06:04 UTC
Updated:
10 Jan 2019 at 08:52 UTC
Jump to comment: Most recent
Comments
Comment #1
drunken monkeyThanks for reporting this issue, and providing a patch!
It seems to me Views PHP works in a rather hacky way, but on the other hand there doesn't seem to be any harm in the adaption you suggest – and if Search API Views then works with that module, why not?
In any case, please re-roll the patch according to this handbook page, so that it can be cleanly applied with Git, and then set the issue's status to "needs review".
Comment #2
sergei_brill commentedOn one of my projects the patch fixed problems with views_php. But now I tested on a clean drupal installation and the patch didn't fix the bug. Property old_view is empty. It looks like it is better to create issue for views_php. May be it is possible to find a better way to run php code from fields.
Comment #4
rcodinaComment #5
rcodinaPatch on #0 doesn't work for me. Also, patch on related issue doesn't work too.
Comment #6
legolasboThis issue has not seen activity in over 2,5 years. I am therefore closing this issue to clean up the issue queue. Feel free to re-open and update this issue if you feel this issue is still relevant and of importance.