Closed (fixed)
Project:
Drupal Crowdfunding
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 Mar 2015 at 05:28 UTC
Updated:
11 Mar 2016 at 13:19 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
tabestan commentedThe funding goal should be an integer. Is there a specific use case where you want the goal to be a string?
Comment #2
Yuri commentedNo I dont want it to be a string, but the code does not take care of this error.
Comment #3
joaogarin commentedYes agree there is no reason for string but it should definitely be verified with a simple check and throw a proper message.
Comment #4
Yuri commentedEventually, this should be a currency field. Otherwise the user is forced to use a site wide currency.
Integrating this module https://www.drupal.org/project/commerce_currency
Comment #5
tabestan commentedHere's the check if "Goal" is a positive number.
Comment #6
victorcpereira commentedThe patch #5 works well.
Comment #7
pedrorocha commentedCommited to 1.x