This is a bug with the RDF/XML parser, rather than with the getUri() method.

@njh it's a bit more complicated, actually. It looks like we don't have a way to change graph's baseUri from the parser. So, either we need to introduce setUri method on EasyRdf_Graph or we need to change parsers protocol so that they create a new graph object instead of using the one given to them as parameter.

I think that second option is better. And even more: it should return object of new EasyRdf_Dataset class, which will give us access to one or several EasyRdf_Graph objects. This way it will be possible to properly handle formats which store several graphs in one document (JsonLD, TriG, …)

sorry… I wrote some nonsense

base-uri is not related to the URI of graph, so, yup, this is a bug in parser

Am I right in reading the XML parser code in startState0 as it basically skipping over the root node (as long as it's an <rdf:RDF>), so an xml:base there never really gets looked at?

The other "levels" all try to account for parent XML base settings, but as far as I can tell the one from the root node never gets saved. A simple hack to try to save that data in startState0 doesn't work, as other parts of the code assume everything in the $sStack has a type/value.

My second stab at a simple solution had better results.

This is the cleanest way I could think of to accomplish this in the existing code, and should respect the root-level xml:base when present while also still using the passed-in base if the root xml:base is absent or a relative URI.

I'm not sure the strategy of modifying $this->xBase is a way you'd like to go, so I didn't make a pull request.

@zerocrates applied your fix. thank you!