Skip to content

Issue #121 Norway Wheelchair equipment fix3 - rebased on Master#123

Closed
nick-knowles wants to merge 9 commits intoNeTEx-CEN:masterfrom
nick-knowles:WheelChairEquipmentFix3
Closed

Issue #121 Norway Wheelchair equipment fix3 - rebased on Master#123
nick-knowles wants to merge 9 commits intoNeTEx-CEN:masterfrom
nick-knowles:WheelChairEquipmentFix3

Conversation

@nick-knowles
Copy link
Contributor

Errors in VehicleType / ActualVehicleEquipment, WheelchairVehicleEquipment , AccessVehicleEquipment

  • WWheelchairEquipmentRef renamed to WheelchairVehicleEquipmentRef
  • Supertypes corrected

Non material chaneg to test workklow
…nd AccessVehicelEquipment

Correct wrong suppertyupes
Corectct inconstisent name for WheechairVehicleEUipmentRef (old name
@skinkie
Copy link
Contributor

skinkie commented Apr 13, 2022

@Aurige @ue71603 this needs a complete review.

The WheelchairVehicleEquipmentRef is already in our NeTEx_publication.xsd but not in our actual files. This pull request changes a lot of things, must be validated.

1 similar comment
@skinkie
Copy link
Contributor

skinkie commented Apr 13, 2022

@Aurige @ue71603 this needs a complete review.

The WheelchairVehicleEquipmentRef is already in our NeTEx_publication.xsd but not in our actual files. This pull request changes a lot of things, must be validated.

<File FilePath="examples\functions\fares\Netex_91.1_Rail_RailCard_MultipleProduct.xml" HomeFolder="Yes"/>
</Folder>
<Folder FolderName="bus">
<Folder FolderName="uk">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest not to update NeTEx.spp

@@ -3,16 +3,8 @@

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest not to update README.md from such an old PR

</xsd:annotation>
<!-- ======================================================================= -->
<!-- ======================================================================= -->
<xsd:complexType name="vehicleEquipments_RelStructure">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already exists and even in an extended form:
<xsd:complexType name="vehicleEquipments_RelStructure">
xsd:annotation
xsd:documentationList of VEHICLE EQUIPMENT.</xsd:documentation>
</xsd:annotation>
xsd:complexContent
<xsd:extension base="containmentAggregationStructure">
<xsd:choice maxOccurs="unbounded">
<xsd:element ref="AccessVehicleEquipment"/>
<xsd:element ref="WheelchairVehicleEquipment"/>
</xsd:choice>
</xsd:extension>
</xsd:complexContent>
</xsd:complexType>
<!-- ====ACCESS=====

</xsd:simpleType>
<xsd:element name="VehicleEquipmentRef" type="InstalledEquipmentRefStructure" substitutionGroup="PassengerEquipmentRef">
<xsd:annotation>
<xsd:documentation>Reference to a WHEELCHAIR VEHICLE EQUIPMENT.</xsd:documentation>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should stay in (if git can find it)

</xsd:element>
<xsd:complexType name="VehicleEquipmentRefStructure" abstract="true">
<xsd:annotation>
<xsd:documentation>Type for a reference to a WHEELCHAIR VEHICLE EQUIPMENT.</xsd:documentation>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should stay in

</xsd:annotation>
<xsd:restriction base="VehicleEquipmentIdType"/>
</xsd:simpleType>
<xsd:element name="WheelchairVehicleRef" type="InstalledEquipmentRefStructure" substitutionGroup="VehicleEquipmentRef">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is already done.

<xsd:documentation>Reference to a WHEELCHAIR VEHICLE EQUIPMENT.</xsd:documentation>
</xsd:annotation>
</xsd:element>
<xsd:complexType name="WheelchairVehicleRefStructure" abstract="false">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is in master, but differently (different type). Needs to be discussed.
<xsd:element name="WheelchairVehicleRef" type="InstalledEquipmentRefStructure" substitutionGroup="VehicleEquipmentRef">
xsd:annotation
xsd:documentationReference to a WHEELCHAIR VEHICLE EQUIPMENT.</xsd:documentation>
</xsd:annotation>
</xsd:element>

<xsd:include schemaLocation="../netex_genericFramework/netex_accessibility/netex_acsb_accessibility.xsd"/>
<xsd:include schemaLocation="netex_equipmentVehiclePassenger_support.xsd"/>
<xsd:include schemaLocation="netex_vehicleType_support.xsd"/>
<xsd:group name="VehicleEquipmentChoice">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still there in master, but used nowhere. So we could remove it.

</xsd:complexContent>
</xsd:complexType>
<!-- ====ACCESS=================================================================== -->
<xsd:element name="ActualVehicleEquipment" type="ActualVehicleEquipment_VersionStructure" abstract="true" substitutionGroup="PassengerEquipment">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still in master

<!-- ====ACCESS=================================================================== -->
<xsd:element name="ActualVehicleEquipment" type="ActualVehicleEquipment_VersionStructure" abstract="true" substitutionGroup="PassengerEquipment">
<!-- =========== ACCESS VEHICLE EQUIPMENT =========================================== -->
<xsd:element name="VehicleEquipment" substitutionGroup="PassengerEquipment">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

new to master. may have impact on other things

<xsd:element name="VehicleEquipment" substitutionGroup="PassengerEquipment">
<xsd:annotation>
<xsd:documentation>An item of EQUIPMENT of a particular type actually available in an individual VEHICLE.</xsd:documentation>
<xsd:documentation>Specialisation of VEHICLE EQUIPMENT for ACCESS providing information such as low floor, ramp, access area dimensions, etc.</xsd:documentation>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't it be AccessVehicleEquipment in that case?

<xsd:group ref="AccessVehicleEquipmentAdditionalGroup"/>
</xsd:sequence>
</xsd:group>
<xsd:group name="AccessVehicleEquipmentGroup2">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removing this makes sense

<xsd:enumeration value="naturalGas"/>
<xsd:enumeration value="biodiesel"/>
<xsd:enumeration value="electricity"/>
<xsd:enumeration value="electricContact"/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changing things here may break the whole thing.

@ue71603 ue71603 requested review from Aurige and trurlurl April 14, 2022 11:54
@ue71603
Copy link
Contributor

ue71603 commented Apr 14, 2022

I think @Aurige needs to check as well. Some parts need to be removed from the PR and the rest could then be submitted.

</xsd:extension>
</xsd:complexContent>
</xsd:complexType>
<xsd:group name="ActualVehicleEquipmentGroup">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also different in master.

@ue71603
Copy link
Contributor

ue71603 commented Nov 21, 2023

@Aurige Also here: Can you mark what you want to keep and @trurlurl or I will shrink the PR to the real delta.

@Aurige
Copy link
Contributor

Aurige commented Nov 22, 2023

It looks that we already have what we need ... and there are breaking changes here... I would rather forget this PR
I will add it to the discussion on Monday to make sure.

<xsd:restriction base="VersionOfObjectRefStructure">
<xsd:attribute name="ref" type="ActualVehicleEquipmentIdType" use="required">
<xsd:annotation>
<xsd:documentation>Identifier of a ACTUAL VEHICLE EQUIOMENT.</xsd:documentation>
Copy link
Contributor

@skinkie skinkie Dec 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<xsd:documentation>Identifier of a ACTUAL VEHICLE EQUIOMENT.</xsd:documentation>
<xsd:documentation>Identifier of an ACTUAL VEHICLE EQUIPMENT.</xsd:documentation>

ue71603 and others added 2 commits December 18, 2023 10:40
…_support.xsd

Co-authored-by: Stefan de Konink <stefan@konink.de>
…_support.xsd

Co-authored-by: Stefan de Konink <stefan@konink.de>
@Aurige
Copy link
Contributor

Aurige commented Dec 20, 2023

As said before, it's an old PR that is already mostly covered... I would close this one

The only good thing looks to be the
<xsd:element name="actualVehicleEquipments" type="actualVehicleEquipments_RelStructure" minOccurs="0">
instead of
<xsd:element name="actualVehicleEquipments" type="equipments_RelStructure" minOccurs="0">

But since an actualVehicleEquipment is an equipment that's just adding a constraint, not adding a new feature. We may to a separated PR for this.

ue71603 added a commit that referenced this pull request Dec 21, 2023
@ue71603 ue71603 mentioned this pull request Dec 21, 2023
@ue71603
Copy link
Contributor

ue71603 commented Dec 21, 2023

rest done #622

@ue71603 ue71603 closed this Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants