atlasmap-attic / atlasmap-runtime

AtlasMap runtime engine and design time services
http://atlasmap.io
Other
3 stars 8 forks source link

AtlasService.converterCheck() contains incomplete code #273

Closed jpav closed 6 years ago

jpav commented 6 years ago

Code near the end of the method that determines whether a response should be generated indicating a converter is needed to perform a mapping has been left incomplete. We should determine the necessity of this code and either complete its implementation or remove it from the class.

igarashitm commented 6 years ago

Weird thing is that we sometimes see "converter is not available" warning in UI. I thought it came from this converterCheck. Need to investigate.

igarashitm commented 6 years ago

I now suspect the converterCheck is not used at all. All the converter validation results I have ever seen come from validate. Make sure UI doesn't invoke it and remove.

igarashitm commented 6 years ago

Issue moved to atlasmap/atlasmap #14 via ZenHub