Closed (fixed)
Project:
Question/Answer
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
30 Apr 2012 at 19:04 UTC
Updated:
16 Sep 2012 at 20:11 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jcisio commentedI think "duration" is optional (if it is not, then it's a bug).
About best answer feature : how it could be optional, because it's the core feature?
Comment #2
nevets commentedDuration is not optional, it's required. And question/answer works as a useful module even without best answer (the request comes indirectly from a client who did not want the best answer part)
Comment #3
Anonymous (not verified) commentedI'm here, because I wondered what the duration is for and if it is required at all. Nevets, I agree, best answer is nice, but a question can have many answers and even many good among them. So it should be optional.
Expiration actually is optional already. It's disabled by setting to zero. This should be a configuration option (rather than requiring such a field, hiding it and setting it to zero). In config, it is required... (screenshot)
Anyway, this issue is at very low priority for me, too. Just because there are quite obvious workarounds and the simplicity of the module at the current state is appreciated (few space for failure).
Don't forget about #1505098: Undefined Index in question_answer.pages.inc with 7.x-1.x-dev to spare yourself the expiration field. It's in the current dev version only.
Comment #4
Anonymous (not verified) commentedIn the code, to me it seems there is nothing about expiration done at all, as long as the Vote API module isn't installed.
There is a great need for clarification! Some have the Vote API, some don't, and just by accident it sometime regards the expiration or not. I tend to say (and correct my previous statement) this is a bug! Because you can't use the module reliably without reading the code and setting up fields manually. So, if it's not a bug, it's simply not production quality.
Comment #5
jcisio commentedComment #6
jcisio commentedOk I revisit this issue and I agree that it is a bug that the expiration (duration field) is required. However, I don't see any reason that the answer field should be optional.
Comment #7
jcisio commentedFixed with:
http://drupalcode.org/project/question_answer.git/commit/1bce34a (6.x)
http://drupalcode.org/project/question_answer.git/commit/d1a70cf (7.x)