Ticket #17 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

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

Screen shot 2010-10-06 at 20.44.20.png Download (95.8 KB) - added by Tuinslak 3 years ago.

Change History

comment:1 Changed 3 years ago by Tuinslak

  • Description modified (diff)

comment:2 Changed 3 years ago by Tuinslak

  • Description modified (diff)

comment:3 Changed 3 years ago by Tuinslak

Issue 1 fixed with .htaccess => AddDefaultCharset UTF-8

Last edited 3 years ago by Tuinslak (previous) (diff)

comment:4 Changed 3 years ago by Tuinslak

  • Description modified (diff)

comment:5 Changed 3 years ago by Tuinslak

  • Description modified (diff)

comment:6 Changed 3 years ago by Tuinslak

  • Description modified (diff)

comment:7 Changed 3 years ago by Tuinslak

Issue 2 fixed with correct $_COOKIE instead of $_POST

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:12 Changed 3 years ago by Tuinslak

  • Description modified (diff)

comment:13 Changed 3 years ago by Tuinslak

  • Description modified (diff)

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.

comment:15 Changed 3 years ago by pietercolpaert

  • Description modified (diff)

comment:16 Changed 3 years ago by Tuinslak

  • Description modified (diff)

Changed 3 years ago by Tuinslak

Note: See TracTickets for help on using tickets.