| Age | Commit message (Collapse) | Author | |
|---|---|---|---|
| 2018-02-22 | WorkbenchImportWorker: Use temp files for entry group subdir Zips | Teddy Wing | |
| Instead of creating our own files for these subdirectory Zip files, use temp files. There are two advantages to doing this: 1. We can customise the location they get saved to. This means we can use `Rails.root` as a default, but specify a custom location in a Rails config variable. 2. The files will be automatically deleted on every garbage collection. In actuality, #2 above was already handled by the `eg_file.unlink` line, and #1 could be easily added to the existing code. Using a `Tempfile`, though, makes the fact that these are temporary files more explicit. The reason why we want to be able to customise the location of these tempfiles is that in our production environment set up by Puppet, the `#{Rails.root}/tmp/imports` directory isn't writable by the application, which causes this code to error. We thus want a way to change the path in different environments. We keep the `eg_file.unlink` call here because we want the temporary file to be deleted as soon as possible. The Ruby documentation even recommends it (https://ruby-doc.org/stdlib-2.5.0/libdoc/tempfile/rdoc/Tempfile.html#class-Tempfile-label-Explicit+close): > it's good practice to do so: not explicitly deleting unused Tempfiles > can potentially leave behind large amounts of tempfiles on > the filesystem until they're garbage collected. I got rid of the `#upload_entry_group_tmpfile` method in this change, moving its contents to `#upload_entry_group_stream`. This was easy enough to do since it was only called in that one place. The reason I did this is because it took an open `File` object as an argument and closed the file at the end of the method. That's a major side effect. I don't think the method should be closing a file it doesn't own. After moving this work to `#upload_entry_group_stream`, we can close and unlink the file in the same place where it was created. Refs #4315 | |||
| 2018-02-22 | Refs #5999 Update workgroup call in TimeTable#new | cedricnjanga | |
| 2018-02-21 | Refs #5942; Apply users TZ when computing a schedule5942-apply-user-tz | Zog | |
| 2018-02-21 | Refs #5944; Skip automated times calculation when no start time is set5944-skip-schedule-calculation-when-no-time-is-set | Zog | |
| 2018-02-21 | Move importers outputs in logs | Zog | |
| 2018-02-21 | Improve output of importers | Zog | |
| 2018-02-21 | Avoid error in referentials/_overview when one of lines isn't associated to ↵ | Alban Peignier | |
| a transport mode | |||
| 2018-02-21 | Merge branch '5865-validate-referential-cloning' of ↵5865-validate-referential-cloning | Luc Donnet | |
| github.com:AF83/stif-boiv into 5865-validate-referential-cloning | |||
| 2018-02-21 | Fix specs | Zog | |
| 2018-02-21 | Refs #5865 @1h; Ensure user is allowed to duplicate a referential before ↵ | Zog | |
| doing so I also changed the way 403 errors are handled, to properly respond with a 403 HTTP code | |||
| 2018-02-20 | Refs #5979 Change export CSV filenames to have a readable date in it5979_change_export_csv_filenames | cedricnjanga | |
| 2018-02-20 | Refs #5882 Fix CC name and code filter5882_ccset_code_filter | cedricnjanga | |
| 2018-02-20 | Refs #5609 Change slug format | cedricnjanga | |
| 2018-02-20 | Use page as will_paginate param_name by default. Refs #3542 | Alban Peignier | |
| 2018-02-20 | Merge pull request #304 from af83/5901-fix-calendar-autocompletion | teddywing | |
| 5901 Makes TimeTable search accent-insensitive | |||
| 2018-02-20 | Refs #5901; Update specs and fix unaccented search | Zog | |
| 2018-02-20 | Refs #5901; Makes TimeTable search accent insensitive | Zog | |
| 2018-02-20 | Merge pull request #308 from ↵ | Alban Peignier | |
| af83/5911-stops-filter-on-referential-journeys-index Better stops filters on ReferentialVJs#index. Refs #5911 | |||
| 2018-02-20 | Merge pull request #297 from af83/5486-fix-react-warnings | Luc Donnet | |
| 5486 Use minified version of React/Redux in production | |||
| 2018-02-20 | Merge pull request #294 from af83/5863-use-shallow-urls-for-referentials | Luc Donnet | |
| 5863 Remove workbench id from the querystring | |||
| 2018-02-20 | Fix specs on VehicleJourneyAtStop | Zog | |
| 2018-02-20 | Merge branch '5863-use-shallow-urls-for-referentials' of ↵ | Luc Donnet | |
| github.com:AF83/stif-boiv into 5863-use-shallow-urls-for-referentials | |||
| 2018-02-20 | Refs #5863 @6h; Remove workbench id from the querystring | Zog | |
| Infer it when possible, and use a nested otherwise | |||
| 2018-02-20 | Fix specs | Zog | |
| 2018-02-20 | Refs #5924; Fix SimpleImporter lookup scope | Zog | |
| 2018-02-20 | Regs #5924; Rename imports tasks | Zog | |
| 2018-02-20 | Refs #5924; Remove `local_id` methods | Zog | |
| 2018-02-20 | Refs #5924; Fix some bugs revealed during imports | Zog | |
| - The checksum computing for VehicleJourneyAtStops - Offsets calculation in VehicleJourneys | |||
| 2018-02-20 | Refs #5924 @14h; Extend importers | Zog | |
| Mostly add a way to override the default behaviour and process each row its own way | |||
| 2018-02-20 | Refs #5924 @2h; Update specs | Zog | |
| 2018-02-20 | Refs #5924; Update rake tasks | Zog | |
| 2018-02-20 | Refs #5924 @2h; Provide a mechanism to define a custom importer | Zog | |
| 2018-02-20 | Refs #5924 @1H; Add `before` actions | Zog | |
| 2018-02-20 | Fix specs | Zog | |
| 2018-02-20 | Refs #5765; :fire: comment | Zog | |
| 2018-02-20 | Refs #5765; Add new task to import in a given referential | Zog | |
| 2018-02-20 | Refs #5765 @6h; Add a customizable importer mechanism | Zog | |
| 2018-02-20 | Replace current_workgroup by referential.workgroup. Refs #5867 | Alban Peignier | |
| 2018-02-19 | fix type in secret file for docker | Florent Peyraud | |
| 2018-02-19 | Remove ReferentialVehicleJourneysController when only end stop. Refs #59115911-stops-filter-on-referential-journeys-index | Alban Peignier | |
| 2018-02-19 | Search VehicleJourneys with one StopArea or two ordered StopAreas. Refs #5911 | Alban Peignier | |
| 2018-02-19 | Fixes StopArea#start/end order. Refs #5911 | Alban Peignier | |
| 2018-02-19 | Refs #5911 @2h; Better stops filters on ReferentialVJs#index | Zog | |
| 2018-02-19 | Merge pull request #303 from af83/5899-use-waiting-time-in-schedules | Alban Peignier | |
| Use waiting_time in schedules computation. Refs #5899 | |||
| 2018-02-19 | Merge pull request #309 from af83/5919-make-coluns-sortable | Alban Peignier | |
| Make ReferentialVJs#index sortable. Refs #5919 | |||
| 2018-02-19 | Merge pull request #286 from af83/5853-detailed-calendar-view-in-vjs-index | Alban Peignier | |
| Add a detailed calendars view on VJs editor. Refs #5853 | |||
| 2018-02-19 | Fixes javascript VehicleJourneys snapshot. Refs #58535853-detailed-calendar-view-in-vjs-index | Alban Peignier | |
| 2018-02-19 | Refs #5853; Better css | Zog | |
| 2018-02-19 | Refs #5853 @2h; Add a detailed calendars view on VJs editor | Zog | |
| 2018-02-19 | Avoid error in referentials/_overview when one of lines isn't associated to ↵ | Alban Peignier | |
| a company | |||
