The latest version of Views 2 (the HEAD version after RC1) has broken our Views integration. See http://angrydonuts.com/attention-views-api-developers
We need to implement the new hooks and split our views integration into separate files. Perhaps it'd be a good time to create an "includes" directory also, as we're accumulating quite a few of them now.
Comments
Comment #1
mooffie commentedI wasn't aware of this. I'll look into this later.
(Yes, we're starting to have a jungle there.)
Comment #2
mooffie commentedWe want to have at least one beta that's compatible with the official Views release.
So this issue isn't a beta breaker. I'm demoting this to 'normal'.
Comment #3
mooffie commentedI see Merlin has just released an RC2. So this is back to 'critical'.
Comment #4
mooffie commentedI see Merlin has just released an RC2. So this is back to 'critical'.
Comment #5
mooffie commentedAfter checking the status of various modules, I decided it's best to release Beta4 before upgrading our Views support.
We have conservative users who use our "official" releases. So they're using "official" releases of other modules too, and, since many of these don't support Views' RC2 yet, they're going to stick with RC1 for a bit.
Comment #6
mooffie commentedWhat about the files layout?
You aren't here these days, so I can't ask you for your preferences.
I think one good choice is to have a 'flag_views' folder --as if our Views support were a separate module-- and in it have a 'handlers' folder, and perhaps a 'views' folder to put the default views in (a la Organic Groups).
But perhaps you'll consider this too revolutionary. Then I could put the Views support in 'includes/views' and 'includes/views/handler' (a la CCK). But perhaps you'll consider the extra 'views' folder unneeded.
So I'll put it directly in the 'includes' folder.
Comment #7
mooffie commentedA related issue:
#309076: Have an 'includes' folder
Comment #8
mooffie commentedFixed.
http://drupal.org/cvs?commit=140360
http://drupal.org/cvs?commit=140361
I've added three lines at the bottom of flag.views.inc to makes it work with the old version of Views as well.
Comment #9
chadchandler commentedI can confirm that this is fixed.
Also, I think it's time to get a tip-jar going for Moofie ...
Comment #10
quicksketchAgreed :D
Comment #11
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.