Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Might already be on the list, but couldn't find an issue for the 7.x version.
Comments
Comment #1
rjerome CreditAttribution: rjerome commentedI have a feeling this one may be related to the version of Views or Ctools you are using. Try the latest -dev versions of both.
Comment #2
Milan0 CreditAttribution: Milan0 commentedTested with latest dev of: ctools, panels, views today.
Confirmed not working for these versions :(
Comment #3
rjerome CreditAttribution: rjerome commentedStrange, because they seem to be working for me. Could you be more specific in terms of the error message, and the steps to produce it.
Ron.
Comment #4
Milan0 CreditAttribution: Milan0 commentedCurrently installed modules:
Reverted theme back to original, removed/ added Keyword ID argument.
Added field "Biblio: All keywords All keywords" to default. < works
Added Filter "Keyword ID" to filters < works
No Relationships defined.
Hope this can help, otherwise let me know what more information do you need.
I see nothing funny with the code, but never programmed a view, so i might be overlooking it :/
Comment #5
Milan0 CreditAttribution: Milan0 commentedMaybe i'm missing views logic here, but i thin k "biblio_handler_argument_many_to_one" is just simply missing (?)
Comment #6
Milan0 CreditAttribution: Milan0 commentedYeah, i see it in the 6.x branch, missing in 7.x
Not comitted to CVS ?:)
Comment #7
Milan0 CreditAttribution: Milan0 commentedJust wondering if this can be confirmed, or it's still just an issue with my configuration?
Trying to get the views to work.
Views module is looking for biblio_handler_argument_many_to_one i think.
Devel is giving me this output in views:
Comment #8
rjerome CreditAttribution: rjerome commentedAny chance you could export the view and either post it here or send it to me via email.
Ron.
Comment #9
Milan0 CreditAttribution: Milan0 commentedSure,
I've send you a detailed e-mail!
Comment #10
rjerome CreditAttribution: rjerome commentedAhh, I see the problem now. Some code from the 6.x-1.x branch got merged into the 7.x branch, but the associated handlers did not :-(
I will rectify that.
Ron.
Comment #11
Milan0 CreditAttribution: Milan0 commentedYes, i thought something like that :)
I'll await the update, thanks.
Comment #12
rjerome CreditAttribution: rjerome commentedI also just noticed that you are using the canned "biblio_views" view, you don't have to (and probably shouldn't) use that. You can just add a new view and go from there. I thought I had removed that default "biblio_views" from the distribution, because it can cause some negative interactions with the standard (non-views) operation.
Ron.
Comment #13
rjerome CreditAttribution: rjerome commentedI just committed changes that should resolve this issue
http://drupal.org/cvs?commit=505004
Be warned though, the Drupal infrastructure team is about to convert to GIT (http://drupal.org/node/1068664) so this is not likely to be available immediately ...
Comment #14
Milan0 CreditAttribution: Milan0 commentedThe argument handler code is not 7.x compliant.
There is no more db_placeholders (RIP :)) and you should use foreach on the result instead of the fetch object method.
Comment #15
Milan0 CreditAttribution: Milan0 commentedI tested it with the code i send you through e-mail for the agument handler, and after i removed/ re-added the keyword ID argument, everything works smooth.
Haven't tested with AuthorID, but asuming this would work aswel.
Comment #16
rjerome CreditAttribution: rjerome commentedSorry, I obviously didn't test it extensively. That was quick and dirty last night to get it checked in before the big GIT shutdown.
Ron.
Comment #17
Milan0 CreditAttribution: Milan0 commentedGIT got shutdown already ? :) just kidding, ... i would create a patch for you but i'm on a windows machine :(
marking this issue as to review
Milan
Comment #18
Milan0 CreditAttribution: Milan0 commentedComment #19
Liam MorlandNeeds patch.