Closed (fixed)
Project:
Views Date Range Filter
Version:
5.x-1.6
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Feb 2008 at 03:16 UTC
Updated:
10 Mar 2008 at 11:33 UTC
Jump to comment: Most recent file
Comments
Comment #1
douggreen commentedThanks! Is the
ensure_tablereally needed? I thought that all views were based on nodes.Comment #2
neochief commentedYes it is required, because query builds like "...select * from prefix_node as node..." (or something like that) and your code in WHERE will be "prefix_node.created" , but not "node.created" as it should be
Comment #3
douggreen commentedSo what your saying is
{node}.createdisn't expanded with the database prefix. If that's the case, then I think the solution is to expand it here rather than including the node table again.Does my attached patch solve your problem?
Comment #4
neochief commentedIve tried, but, nope, it still wrong.
here is the full buggy query with your patch
as you see, your module puts "m2b_node.created", but it must put "node.created" to be fine
Comment #5
douggreen commentedHmm, then it's possible that this is the patch needed.
Comment #6
neochief commentedMaybe you are right, but I saw other modules putting ensure_table() in this situation anyway. Actually I do not know what it's doing :) It's just works for me. And without it too.
Comment #7
douggreen commentedI'm not completely sure what you just said. Does the patch in #5 solve your problem? If so, then I'll commit it to th e project.
Comment #8
neochief commentedYes, it solved the problem.
Comment #9
douggreen commentedThanks, committed
Comment #10
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.