search

Roomify / bat_drupal

Booking and Availability Management Tools for Drupal 7 and Drupal 8
33 stars 7 forks source link

Convert Data field to UNIX Timestamp #34

Closed derimagia closed 7 years ago

derimagia commented 8 years ago

The events fields are using 'Date' fields instead of using an int UNIX Timestamp field. What's the main reason for this? It goes against Drupal's standard of fields.

I'll look into converting them if there isn't a good reason.

acrollet commented 8 years ago

@derimagia I'm punting on this one - using datetime represents a change from how we did things in Drupal Rooms, so there was a conscious decision, don't know the reasoning behind it though. Assigning @istos to comment when he's able.

derimagia commented 8 years ago

I'm pretty much finishing up the project I was on that used BAT - This actually caused a lot of issues on my end because of timezones - BAT doesn't seem to convert them to UTC time and stores the info in the database as php's default timezone. If I had to redo it again and I choose to re-use BAT I would convert it to UNIX timestamps at the beginning of the project.