Unable to save with date fields?

patchak - May 28, 2009 - 13:22
Project:Views Saved Searches
Version:6.x-1.0-beta1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

Hey there, I'm using this succesfully for a while now on a site of mine, but I just found a bug.

It seems that the module is not able to save views searches with date module's exposed filters? Would there be any reason that it would work for some exposed filters and not for others? Any idea where to start looking at this?

Thanks,
Patchak

#1

avivalma - May 30, 2009 - 14:27
Status:active» needs review

Date produces a little bit strange views query parameters. Here is a patch what handles them.
This seems to be date specific (true), but if other modules produce the same style query param, it will handle.

AttachmentSize
views_savedsearches_and_date.patch 1.68 KB

#2

patchak - May 30, 2009 - 14:29

Whoooo thanks a lot, testing this tonight!!

Patchak

#3

avivalma - May 31, 2009 - 18:48

Here is a new patch. It works with min / max multiple choice date fields as well. (and it's much simplier as well)

AttachmentSize
views_savedsearches_and_date.patch 1.57 KB

#4

patchak - May 31, 2009 - 19:18

I confirm this patch solved a bug I had when trying to save a search on a view with multiple exposed date filters.

#5

avivalma - May 31, 2009 - 19:36

eliminate error messages when saving the the search. (related to redirections)

AttachmentSize
views_savedsearches_and_date.patch 2.87 KB

#6

pydubreucq - October 15, 2009 - 12:26

Hi,
Excuse-me I'm French and I don't speak english very well...
I apply this patch, but when I click to "Save this Search" My website appears in the div where I must found the saved search :(
I don't understant why, when I reload the entire page, this disapear...
You can see on my screenshot :
Thanks for all...
Bye

#7

pydubreucq - October 15, 2009 - 12:31

Excuse me I've forgotten the screenshot ;)

AttachmentSize
screenshot.png 168.81 KB
 
 

Drupal is a registered trademark of Dries Buytaert.