Class for XML configuration not placed in the correct module.

Description

In CONFOUND-2, the classes XmlConfiguration and XmlConfigurationFileFormat are in the submodule config-file-format-xml, when, to be consistent, it should have the class XmlConfiguration on another new submodule called config-xml.

Environment

None

Activity

Show:

Magno Nascimento October 25, 2018 at 2:39 PM
Edited

I see, okay then, I'll mark it as CLOSED. Thank you for the explanation!

Garret Wilson October 25, 2018 at 2:34 PM

No, the Properties class is something sort of special. It exists as part of Java, and it's somewhat separate from its file format (even though they built serialization into the class, which wasn't a good idea—it's an old class).

Put another way, we agree that config-file-format-properties should be a separate module for the format. But for the configuration that uses Properties, I integrated it into the main configuration class because Properties is such an integral part of the Java standard library. It comes built into Java.

Magno Nascimento October 25, 2018 at 2:27 PM

And what about config-properties? It seems the same thing as the XML stuff.

Garret Wilson October 25, 2018 at 2:25 PM

Ah, I see. But in this case I don't think you need to do that. The only thing using XmlConfiguration is the XML file format, and we can always refactor it in the future if we need it—but I don't think we'll need it.

The reason I created a separate config-urf submodule with UrfConfiguration is that there will be several formats (TURF, CURF, 3URF, etc) that all use an UrfConfiguration because they will parse to an URF graph. But I don't think that applies to the XML situation.

Magno Nascimento October 25, 2018 at 2:17 PM

I'm saying that instead of creating three submodules:

I created only two:

Won't Fix

Details

Assignee

Reporter

Priority

Created October 25, 2018 at 2:10 PM
Updated October 25, 2018 at 2:40 PM
Resolved October 25, 2018 at 2:39 PM