Closed (fixed)
Project:
Pushtape Features
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
1 Dec 2011 at 09:53 UTC
Updated:
6 Aug 2012 at 20:21 UTC
what is the reasoning behind using a text field?
Comments
Comment #1
zirafa commentedA few reasons:
Comment #2
gusaus commentedI don't think date module is going to add a layer of complexity. Especially if it's already a dependency with other features such as #1263956: Events.
Comment #3
zirafa commentedGus, *any* added contrib module adds complexity to the interface, to the backend, and to updating. Every additional module means worrying about any security problems in that module, being dependent on their release cycle, etc.
I'm familiar with the date module and it is very powerful and *does* add a layer of complexity. It has more advanced form field options, stores data differently, formats data for display differently, and includes a Date API to interact with other modules such as Calendar. Just because it is an easy install or works well doesn't mean it's not complex.
Your point about dependencies has me thinking maybe the Release Date should be removed entirely from the base discography feature and moved or merged into a separate event feature.
Comment #4
discipolo commenteddoesnt using the date module depend on whether we want to provide sorting, filtering and (faceted) searching on release dates?
i am sure adding release date field via an aditional feature (app) is an approach worth exploring.
we should maybe get some example artists to enter content to see what they need.
Comment #5
zirafa commentedIt'd be nice to be able to store a simpler default date field that would be compatible with the Date API and could then be used if Date module was needed. Something like http://drupal.org/project/date-time-field would be nice.
Comment #6
ebieymjunior commentedI also think that it's a good idea to make the release date a real date, but won't it be better to just use the date api module than to use another module? Unless you're sure that it'll be updated and ported to future versions when needed.
I installed the Pushtape Features module and I must say it's a time saver for if you want to create a discography feature (instead of manually creating a new content type). I did however delete the release date field and replaced it with a date field. Hopefully it won't complicate things when upgrading.
Thanks for your work, it's really appreciated.
Comment #7
zirafa commentedI've been mulling this over some more and starting to warm to the idea. Date/Date API does seem like the right way to do this and avoid future problems (I looked a little closer at date-time-field and don't think it's a good route.) I think I'd be willing to make this a date field if we could form alter or modify the input field to simplify it so it isn't so overwhelming.
For now I might just remove the date field entirely from this Feature to avoid any confusion...
Comment #8
zirafa commentedRelease Date field has been removed in drupal.org/project/pushtape_discography feature.
Please open a new issue in that queue regarding adding the field back as a proper date field.