Complex opening times impossible to edit #1247

Closed
opened 2021-09-04 12:27:35 +00:00 by arnaudvergnet · 6 comments
arnaudvergnet commented 2021-09-04 12:27:35 +00:00 (Migrated from github.com)

The garden Jardin du sacré coeur in Toulouse, France, has some complex opening times to account for the season.

Organic maps is unable to correctly display this information and the editing screen is empty.

I don't know OSM well enough to tell if the date format is valid, but the app should at least let the user edit the time to correct it if the entry is invalid.

Here is the format:

Nov-Jan 07:45-18:00; Feb 07:45-18:30; Mar 1-15 07:45-19:00; Mar 16-31 07:45-19:30;Apr 07:45-20:00;May 07:45-20:30;Jun-Aug 07:45-21:00;Sep 1-15 07:45-20:30; Sep 16-30 07:45-19:30;Oct 1-15 07:45-19:00;Oct 16-31 07:45-18:30

And here is a video:

https://user-images.githubusercontent.com/80701113/132094410-45c2db1b-2414-4d21-89ce-a619b264340f.mp4

System

  • OS: Android 9
  • Organic Maps version: 2021.08.14-4-F-Droid
  • Data Version: 210729
The garden *[Jardin du sacré coeur](https://www.openstreetmap.org/way/60091397)* in Toulouse, France, has some complex opening times to account for the season. Organic maps is unable to correctly display this information and the editing screen is empty. I don't know OSM well enough to tell if the date format is valid, but the app should at least let the user edit the time to correct it if the entry is invalid. Here is the format: ``` Nov-Jan 07:45-18:00; Feb 07:45-18:30; Mar 1-15 07:45-19:00; Mar 16-31 07:45-19:30;Apr 07:45-20:00;May 07:45-20:30;Jun-Aug 07:45-21:00;Sep 1-15 07:45-20:30; Sep 16-30 07:45-19:30;Oct 1-15 07:45-19:00;Oct 16-31 07:45-18:30 ``` And here is a video: https://user-images.githubusercontent.com/80701113/132094410-45c2db1b-2414-4d21-89ce-a619b264340f.mp4 # System - **OS:** `Android 9` - **Organic Maps version:** `2021.08.14-4-F-Droid` - **Data Version:** `210729`
biodranik commented 2021-09-04 17:45:30 +00:00 (Migrated from github.com)

Thanks! There should be some bug in our code. Opening Hours IsValid says that it's ok though…

Thanks! There should be some bug in our code. Opening Hours IsValid says that it's ok though…
arnaudvergnet commented 2021-09-05 18:51:21 +00:00 (Migrated from github.com)

What's weirder is that I get a blank window, but there seems to be some TextEdit components in the screen as the keyboard opens when I click

What's weirder is that I get a blank window, but there seems to be some TextEdit components in the screen as the keyboard opens when I click
kudlav commented 2021-09-09 10:44:28 +00:00 (Migrated from github.com)

Hi,
I am facing to the same issue with the expert mode.
Even when I try to edit simple opening hours Mo-Su 11:00-24:00 or POI without opening hours.

  1. Open POI with or without opening hours
  2. Edit, Edit opening hours
  3. Make the input box blank
  4. Type "c"
  5. The ui glitch appears
  6. You can continue typing and when you type "closed", it turns back into normal UI and the value can be saved.

So it works but cannot see what typing.

logs.zip
Screenshot_20210909-123825_Organic Maps
Screenshot_20210907-121318_Organic Maps

Hi, I am facing to the same issue with the expert mode. Even when I try to edit simple opening hours `Mo-Su 11:00-24:00` or POI without opening hours. 1) Open POI with or without opening hours 2) Edit, Edit opening hours 3) Make the input box blank 4) Type "c" 5) The ui glitch appears 6) You can continue typing and when you type "closed", it turns back into normal UI and the value can be saved. So it works but cannot see what typing. [logs.zip](https://github.com/organicmaps/organicmaps/files/7135843/logs.zip) ![Screenshot_20210909-123825_Organic Maps](https://user-images.githubusercontent.com/11516259/132672054-20e75f5e-f505-4e10-ae88-e2a48189c438.png) ![Screenshot_20210907-121318_Organic Maps](https://user-images.githubusercontent.com/11516259/132672086-1c57b2cf-6bc2-47cd-a636-a0ca0283ec1e.png)
anibyl commented 2021-09-19 13:45:34 +00:00 (Migrated from github.com)

Same issue when trying to add this tag to a place. It says 24/7 by default, any edit breaks the view. Validation should be performed while confirming, I think.

Same issue when trying to **add** this tag to a place. It says 24/7 by default, any edit breaks the view. Validation should be performed while confirming, I think.

Same issue when trying to add this tag to a place. It says 24/7 by default, any edit breaks the view. Validation should be performed while confirming, I think.

I agree. Or just make a red outline around input field if it is invalid (instead of hiding complete input field).

> Same issue when trying to **add** this tag to a place. It says 24/7 by default, any edit breaks the view. Validation should be performed while confirming, I think. I agree. Or just make a red outline around input field if it is invalid (instead of hiding complete input field).
Member

another test case (with only months): Greenwich park

opening_hours = Oct-Feb 06:00-18:00; Mar-Apr,Sep 06:00-20:00; May-Aug 06:00-21:00

signal-2022-11-15-104416_002

another test case (with only months): [Greenwich park](https://www.openstreetmap.org/way/4483968) `opening_hours = Oct-Feb 06:00-18:00; Mar-Apr,Sep 06:00-20:00; May-Aug 06:00-21:00` ![signal-2022-11-15-104416_002](https://user-images.githubusercontent.com/26939824/201900505-a067e3d1-bf48-4511-a656-584dc9a02911.png)
This repo is archived. You cannot comment on issues.
No labels
Accessibility
Accessibility
Address
Address
Android
Android
Android Auto
Android Auto
Android Automotive (AAOS)
Android Automotive (AAOS)
API
API
AppGallery
AppGallery
AppStore
AppStore
Battery and Performance
Battery and Performance
Blocker
Blocker
Bookmarks and Tracks
Bookmarks and Tracks
Borders
Borders
Bug
Bug
Build
Build
CarPlay
CarPlay
Classificator
Classificator
Community
Community
Core
Core
CrashReports
CrashReports
Cycling
Cycling
Desktop
Desktop
DevEx
DevEx
DevOps
DevOps
dev_sandbox
dev_sandbox
Directions
Directions
Documentation
Documentation
Downloader
Downloader
Drape
Drape
Driving
Driving
Duplicate
Duplicate
Editor
Editor
Elevation
Elevation
Enhancement
Enhancement
Epic
Epic
External Map Datasets
External Map Datasets
F-Droid
F-Droid
Fonts
Fonts
Frequently User Reported
Frequently User Reported
Fund
Fund
Generator
Generator
Good first issue
Good first issue
Google Play
Google Play
GPS
GPS
GSoC
GSoC
iCloud
iCloud
Icons
Icons
iOS
iOS
Legal
Legal
Linux Desktop
Linux Desktop
Linux packaging
Linux packaging
Linux Phone
Linux Phone
Mac OS
Mac OS
Map Data
Map Data
Metro
Metro
Navigation
Navigation
Need Feedback
Need Feedback
Night Mode
Night Mode
NLnet 2024-06-281
NLnet 2024-06-281
No Feature Parity
No Feature Parity
Opening Hours
Opening Hours
Outdoors
Outdoors
POI Info
POI Info
Privacy
Privacy
Public Transport
Public Transport
Raw Idea
Raw Idea
Refactoring
Refactoring
Regional
Regional
Regression
Regression
Releases
Releases
RoboTest
RoboTest
Route Planning
Route Planning
Routing
Routing
Ruler
Ruler
Search
Search
Security
Security
Styles
Styles
Tests
Tests
Track Recording
Track Recording
Translations
Translations
TTS
TTS
UI
UI
UX
UX
Walk Navigation
Walk Navigation
Watches
Watches
Web
Web
Wikipedia
Wikipedia
Windows
Windows
Won't fix
Won't fix
World Map
World Map
No milestone
No project
No assignees
3 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: organicmaps/organicmaps-tmp#1247
No description provided.