Closed (fixed)
Project:
Openlayers
Version:
6.x-2.x-dev
Component:
OpenLayers Views
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
31 Mar 2010 at 18:59 UTC
Updated:
7 Oct 2010 at 14:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
zzolo commentedThanks @yhahn. Commited: http://drupal.org/cvs?commit=348772
Comment #3
Will White commentedThis issue has regressed as a result of this commit. New patch attached.
Comment #4
zzolo commentedHi @Will White. Thanks for the patch.
I have not tested this, but your patch is changing the default value to be in the wrong format for the change in the data_source form item to a select, which just takes a string value as its default.
Most of the code in that commit is a workaround to support backwards compatibility for that change to a select, as its values structure is different than the multiple select, which is the format everyone with existing views has.
Have you tested this with an upgrade?
Have you tested this with Views 2?
Overall, just thinking about this, it doesn't seem right. But I could easily be missing something here.
Comment #5
yhahn commentedThis is not true. The options returned by option definition array should be a keyed array where the keys refer to a variety of attributes of that option, one of which is its default value. Please see the base class
views_object()from which all views handlers and plugins are derived:Comment #6
tmcw commentedGreat, committed: http://drupal.org/cvs?commit=426390