Closed (fixed)
Project:
Signup
Version:
7.x-1.x-dev
Component:
Views integration
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Jan 2012 at 23:31 UTC
Updated:
17 Mar 2012 at 04:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
chaloum commentedI've installed to current version on Signup and am still getting an AJAX error when trying to add the signup link to a view.
is there a patch available that can fix this?
Comment #2
jordojuice commentedCan you paste the Views query so we can have a look?
Comment #3
chaloum commented@jordojuice
this is the Ajax error
and theis is the sql
Comment #4
jordojuice commentedAhh I reproduced this. I'll try to come up with a patch for it.
Comment #5
jordojuice commentedAlright. This patch should fix the issue. I have another patch at #1285536: SQL error when using Signup: Enabled/Disabled (Enabled) that fixes a bug in the signup enabled/disabled filter as well if you need it.
Comment #6
jordojuice commented@chaloum Can you test or have you tested this patch? You should be able to paste the few lines if you don't do patches.
Comment #7
jordojuice commentedMarking this as a duplicate. The patch at http://drupal.org/node/1358258#comment-5561056 fixes this and the issue at #1285536: SQL error when using Signup: Enabled/Disabled (Enabled)
Comment #8
sgabe commentedWe should treat this separately and not merge with other issues.
Comment #9
sgabe commentedAdding nid as an additional field eliminates the error message but the field will not show up because of the missing option_definition() the attached patch also adds this.
Comment #10
jordojuice commentedLooks good. Works for me.
+1
Comment #11
chaloum commented@sgabe
when I apply this patch I no longer have the "Signup: Node: Signup link" available. So it isn't working for me
Comment #12
sgabe commented@chaloum: try it on a clean install and apply the patch against the latest version in the repository. However you should recreate and save again your views where you are using this field.
By the way what do you mean exactly by "available"? Can't you see the Signup: Node: Signup link field on the administration page among the others, so you can't add it to a view? Or is it not displayed in the view?
Comment #13
chaloum commentedsgabe, No longer have the "Signup: Node: Signup link" available = was previously there (available) and no longer there (no longer available)
available as in to use, either in the view creation and it follows that it doesn't display in a view either
sgabe Does the Development release contain all the other patches? I'm nervous about doing a fresh install particularly with the Recommended Release as there are so many different of patches covering various issues flying around, I could end up in a worse situation than I am in ATM.
Comment #14
sgabe commented@chaloum: The reason why the patch does not work for you can be that you have applied numerous unverified patches.
The most important thing about bugs is "If you cannot reproduce it, you cannot fix it!". The patch seems to work for jordojuice and me too. So somebody needs to help us reproduce the bug (if there is still a bug) or we will not be able to fix it.
Since you are the one who reported the problem, you should try to reproduce it on a clean install (absolutely clean system with just the needed modules). If the problem persist, try to provide the steps to reproduce it. Reverting the views provided by the module to default may help.
Comment #15
sgabe commentedThe patch seems to be fine. If there is still a problem, it most likely will be something else (and will be another issue), so I went ahead and committed the patch in #9.
Comment #16
generalconsensus commented@chaloum the issue you might be seeing is possibly related to http://drupal.org/node/1460382. It's looking that something needs to be done with the date include file as it currently stands. I'm also still trying to figure out what is going wrong with date/signup and using the signup link here is another link to more info http://drupal.org/node/1452920