Merge pull request #6 from geertw/develop

Versie 1.2.4

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 1.2.4
+
+* Voorbereid op hernoeming vervoerder 'NS Interna' naar 'NS Int'
+
 ## 1.2.3
 
 * Dubbele actuele eindbestemming wordt samengevoegd (#3)

CONTRIBUTING.md

@@ -0,0 +1,31 @@
+Contributing
+============
+
+Deze applicatie is ontwikkeld in Python. Verbeteringen of uitbreidingen zijn van harte welkom! Ook meldingen van foute verwerking of ontbrekende functies zijn bijzonder nuttig.
+
+Dit document beschrijft kort wat de beste manier is om mee te helpen in de ontwikkeling van deze applicatie.
+
+Bugs en feature requests melden
+-------------------------------
+
+Verzoeken om nieuwe functies, meldingen van bugs, onjuiste dataverwerking of andere problemen, en alle andere meldingen kun je doen in de [issue tracker op GitHub](https://github.com/geertw/rdt-infoplus-dvs/issues). Wanneer je een melding doet van onjuiste dataverwerking helpt het bijzonder wanneer je een (fragment) van de XML-brondata uit InfoPlus toevoegt.
+
+Zelf ontwikkelen
+----------------
+
+In GitHub kun je het project forken en zelf de code aanpassen of verbeteren. Aanpassingen, toevoegingen of verbeteringen kun je daarna weer delen door een pull request aan te maken. Zie ook de [documentatie van GitHub](https://help.github.com/articles/using-pull-requests).
+
+Voorbeeldberichten zoals door DVS verspreid worden zijn te vinden in de directory `/testdata/`. Deze kunnen erg nuttig zijn bij het ontwikkelen. Daarnaast is er via het NDOV-loket [documentatie over InfoPlus](https://ndovloket.nl/helpdesk/kb/31/) beschikbaar waarin alle attributen en publicatierichtlijnen beschreven worden.
+
+Let op; alle code komt automatisch onder GNU GPL v3. Ook wanneer je zelf code aanpast of toevoegt die je niet met het hoofdproject deelt ben je volgens de licentie verplicht om de broncode openbaar te maken. Zie ook de [licentie](LICENSE.txt).
+
+Branches
+--------
+
+De **master** branch is altijd een production-ready branch. De laatste stable release is een tag op de master branch.
+
+De **develop** branch is de ontwikkelversie. Kleine bugfixes en toevoegingen worden op de develop branch gemaakt.
+
+Grote aanpassingen of toevoegingen kun je het beste op een separate feature branch maken. Mocht er dan nog klein werk nodig zijn voordat de functie ingepast kan worden, dan kan dat op de feature branch gebeuren.
+
+Deze werkwijze is gebaseerd op de uitstekend beschreven [GIT workflow](http://nvie.com/posts/a-successful-git-branching-model/) van Vincent Driessen.

README.md

@@ -179,15 +179,17 @@ Koppeling met Munin en/of Nagios als monitoringsysteem is mogelijk; zie de direc
 Ontwikkeling
 ============
 
-Deze applicatie is ontwikkeld in Python. Verbeteringen of uitbreidingen zijn van harte welkom! Maak een pull request aan en werk op de **develop** branch in git. De master branch bevat alleen stabiele versies. Werk voor nieuwe functies of grote aanpassingen bij voorkeur op een feature branch.
-
-Voorbeeldberichten zoals door DVS verspreid worden zijn te vinden in de directory /testdata/.
+Deze applicatie is ontwikkeld in Python. Verbeteringen of uitbreidingen zijn van harte welkom! Ook wanneer je niet mee wilt ontwikkelen, maar in het gebruik wel bugs of andere problemen ervaart, kun je meehelpen door een issue aan te maken. Zie het document [Contributing](CONTRIBUTING.md) voor meer informatie.
 
 Licentie
 ========
 
 Deze applicatie wordt vrij verspreid op basis van de GNU General Publice License (GPL) versie 3.
 
+Een belangrijke voorwaarde van de GNU GPL-licentie is dat je de broncode van de applicatie deelt
+wanneer je wijzigingen of toevoegingen maakt op basis van deze applicatie.
+Zie het bestand LICENSE.txt voor de volledige versie van deze licentie.
+
 This program is free software: you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
 the Free Software Foundation, either version 3 of the License, or

infoplus_dvs.py

@@ -45,7 +45,7 @@ def parse_trein(data):
     trein.vervoerder = trein_node.find('{urn:ndov:cdm:trein:reisinformatie:data:2}Vervoerder').text
 
     # Fix voor verkeerde naam 'NS Interna' voor NS International (zie #1):
-    if trein.vervoerder == 'NS Interna':
+    if trein.vervoerder == 'NS Interna' or trein.vervoerder == 'NS Int':
         trein.vervoerder = 'NS International'
     elif trein.vervoerder == 'Locon Bene':
         trein.vervoerder = 'Locon Benelux'
@@ -795,6 +795,7 @@ def oorzaak_engels(self):
 
         vertalingen = {
             'geplande werkzaamheden': 'planned engineering work',
+            'werkzaamheden': 'engineering work',
             'onverwachte werkzaamheden': 'unexpected engineering work',
             'uitgelopen werkzaamheden': 'over-running engineering works',
             'uitloop van werkzaamheden': 'over-running engineering works',
@@ -812,6 +813,7 @@ def oorzaak_engels(self):
             'sein- en wisselstoring': 'signalling and points failure',
             'sein-en wisselstoring': 'signalling and points failure',
             'een sein- en wisselstoring': 'signalling and points failure',
+            'een sein-en wisselstoring': 'signalling and points failure',
             'storing aan bediensysteem seinen en wissels': 'a control system failure',
             'een storing aan bediensysteem seinen en wissels': 'a control system failure',
             'defect materieel': 'a broken down train',
@@ -886,6 +888,7 @@ def oorzaak_engels(self):
             'een brandmelding': 'a fire alarm',
             'een voorwerp in de bovenleiding': 'an obstacle in the overhead wire',
             'Koningsdag': 'King\'s day',
+            'Vierdaagse': 'Vierdaagse (Four Days Marches)',
             'rommel op het spoor': 'rubbish on the track',
             'grote drukte': 'large crowds',
             'blikseminslag': 'lightning'