Ticket #17 (closed bug: fixed)
Mobile v2 site
| Reported by: | Tuinslak | Owned by: | pietercolpaert |
|---|---|---|---|
| Priority: | major | Milestone: | Mobile Site 1.5 |
| Component: | Mobile website | Keywords: | |
| Cc: | Tuinslak |
Description (last modified by Tuinslak) (diff)
Following bugs should be checked:
FIXED - Error in certain chars (Anvers-Berchem [B] => Berchem-Sainte-Agathe [B])
- output:
- IC739
- IC9260
- Mét5
- Bus84
- Bus84
WON'T FIX YET - All stations are in French (using the EN or DE url will respond with French stations). => only needs fixing in station.php
WILL FIX LATER - Would be cool to check the lang cookie, and get Dutch station names if lang is set to NL
WON'T FIX - Try querying "qmsldkfjqmlskjdfmqklsjdfkmlj" as station.
- NMBS picks "liedekerke" as station. Uh?!
- Pretty much any random text results in a station. Crazy regex?
FIXED - Query in the future (change 2010 to 2011):
- Warning: Invalid argument supplied for foreach() in /home/yeri/irail/api/InputHandlers/BRailConnectionInput.php on line 95
- Fatal error: Call to a member function getDepart() on a non-object in /home/yeri/irail/api/OutputHandlers/MobileWebOutput.php on line 25
FIXED- if 0m delay change output to /
- if delay > 0, change minutes to red text ? /discuss
WILL TRY NOT TO FORGET - query_nat.php => rename to results/ when committing (changed)
NOT FIXED - from & to etc are NOT translated in query_nat.php (and _int.php?)
Thanks to Frank Lycops for reporting some of these bugs
Attachments
Change History
comment:8 Changed 3 years ago by Tuinslak
Issue 3: not our issue. Insane regex by NMBS
Issue 4: Error occurs because request doesn't give expected output
- better error controlling
- 2011 time table is not released yet (11th or so Dec 2010)
- see national & international.php, the echo date('Y'); isn't correct at all, should be solved by something consistent
comment:9 Changed 3 years ago by Tuinslak
Issue 5: should be fixed on dev.irail.be
=> waiting for a train with delays
comment:10 Changed 3 years ago by Tuinslak
- Status changed from new to closed
- Resolution set to fixed
issue 5: tested, all ok
comment:11 Changed 3 years ago by Tuinslak
- Status changed from closed to reopened
- Resolution fixed deleted
comment:14 Changed 3 years ago by pietercolpaert
- Status changed from reopened to closed
- Resolution set to fixed
Ticket closed - You're never done creating a new version. Apart for the new layout, everything is in place.

