Merge pull request #64 from europeana/EA-3842-exception-handling-and_…

…logging

need to get service ID before calling the translation for error logging

translation-web/src/main/java/eu/europeana/api/translation/web/service/TranslationWebService.java

@@ -69,10 +69,11 @@ public TranslationResponse translate(TranslationRequest translationRequest)
     String serviceId = null;
     for (TranslationService cachedTranslationService : cachedTranslationServices) {
       try {
+        serviceId = cachedTranslationService.getServiceId();
         // send the values which are not yet translated (isTranslated=false) for the translations
         cachedTranslationService.translate(translObjs.stream().filter(to -> !to.isTranslated()).collect(Collectors.toList()));
-        // call this method after the translate() method, because the serviceId changes depending if
-        // there is sth in the cache
+        // update service ID after the translate() method, because the serviceId may change (depending if there is sth in the cache)
+        //NOTE: is this really needed?
         serviceId = cachedTranslationService.getServiceId();
         // clear translation error if the invocation is successfull
         translationError = null;