OGR LVBAG Driver ignores field "verkortenaam" in OpenbareRuimte feature type

[justb4]

Expected behavior and actual behavior.

@yorickdewid

The LVBAG Driver reads the feature type "openbareruimte" (is like Eng. "thoroughway", usually a "street" with a streetname). The main field is "naam" (name). In some cases this FT contains an abbreviated name, known as "verkorte openbare ruimte naam".
We expect this field to show in the resulting features .e.g. the PostGIS table "openbareruimte".

Below is an example feature XML .

        <sl:stand>
            <sl-bag-extract:bagObject>
                <Objecten:OpenbareRuimte>
                    <Objecten:identificatie domein="NL.IMBAG.Openbareruimte">0221300000311195</Objecten:identificatie>
                    <Objecten:naam>Schout bij Nacht Doormansingel</Objecten:naam>
                    <Objecten:type>Weg</Objecten:type>
                    <Objecten:status>Naamgeving uitgegeven</Objecten:status>
                    <Objecten:geconstateerd>N</Objecten:geconstateerd>
                    <Objecten:documentdatum>2009-01-20</Objecten:documentdatum>
                    <Objecten:documentnummer>BAG/OBR20080001</Objecten:documentnummer>
                    <Objecten:ligtIn>
                        <Objecten-ref:WoonplaatsRef domein="NL.IMBAG.Woonplaats">2142</Objecten-ref:WoonplaatsRef>
                    </Objecten:ligtIn>
                    <Objecten:voorkomen>
                        <Historie:Voorkomen>
                            <Historie:voorkomenidentificatie>1</Historie:voorkomenidentificatie>
                            <Historie:beginGeldigheid>2009-01-20</Historie:beginGeldigheid>
                            <Historie:tijdstipRegistratie>2010-12-15T10:57:31.000</Historie:tijdstipRegistratie>
                            <Historie:BeschikbaarLV>
                                <Historie:tijdstipRegistratieLV>2010-12-15T11:02:21.484</Historie:tijdstipRegistratieLV>
                            </Historie:BeschikbaarLV>
                        </Historie:Voorkomen>
                    </Objecten:voorkomen>
                    <Objecten:verkorteNaam>
                        <nen5825:VerkorteNaamOpenbareRuimte>
                            <nen5825:verkorteNaam>Sbn Doormansingel</nen5825:verkorteNaam>
                        </nen5825:VerkorteNaamOpenbareRuimte>
                    </Objecten:verkorteNaam>
                </Objecten:OpenbareRuimte>
            </sl-bag-extract:bagObject>
        </sl:stand>

As can be seen the abbreviated name is even nested, e.g.

<Objecten:verkorteNaam>
  <nen5825:VerkorteNaamOpenbareRuimte>
     <nen5825:verkorteNaam>Sbn Doormansingel</nen5825:verkorteNaam>
  </nen5825:VerkorteNaamOpenbareRuimte>
</Objecten:verkorteNaam>

Steps to reproduce the problem.

• Read a BAG source XML into any format, e.g. PostGIS.
• Observe that the field "verkorteNaam" is never present in "Openbareruimte" table

Operating system

Any.

GDAL version and provenance

Latest Docker image as on aug 22, 2021.

Proposed Solution

This issue and its solution is very similar to issue #4161 and its PR #4168.
i.e. a new Field definition (String) needs to be added in ogrlvbaglayer.cpp from L289 on, plus tests.
Only complicating factor is the nested definition, so I expect that additional parsing is required in the function
ogrlvbaglayer.StartElementCbk.

...But it looks like OGR parses recursively until a matching element-name is found (?).
So hopefully it is just enough to define the new field as "verkorteNaam" by just adding:

        OGRFieldDefn oField3("verkorteNaam", OFTString);

[justb4]

I can make the PR. Main question remaining: quoting from above:

...But it looks like OGR parses recursively until a matching element-name is found (?).
So hopefully it is just enough to define the new field as "verkorteNaam" by just adding:

OGRFieldDefn oField3("verkorteNaam", OFTString);

Would that be enough to extract from this element field XML-construct?

<Objecten:verkorteNaam>
  <nen5825:VerkorteNaamOpenbareRuimte>
     <nen5825:verkorteNaam>Sbn Doormansingel</nen5825:verkorteNaam>
  </nen5825:VerkorteNaamOpenbareRuimte>
</Objecten:verkorteNaam>

Could anyone confirm? @yorickdewid or @rouault ?