Add locked property to Port and enhance port sorting logic

[achrafmohye]

Close #634

Description

Port.locked marks a port position as user defined and immutable, automatic sorting ignore locked ports and arrange only the remaining ones around them, while preserving the locked layout across import, export

Issues

Checklist

• This PR contains a description of the changes I'm making
• I've read the Contribution Guidelines
• I've added tests for changes or features I've introduced
• I documented any high-level concepts I'm introducing in documentation/
• CI is currently green and this is ready for review

[aiAdrian]

Please add short commeng for what this member 'locked' is used for

[achrafmohye]

Done :)

[Synar]

As I sent you before, you can simplify by dropping portsToKeepLocked doing something like :

  sortPorts() {
    const originalPorts = this.ports;
    const lockedIndexes: Set<number> = new Set();
    const portsToReorder: Port[] = [];

    originalPorts.forEach((port, index) => {
      if (port.getLocked()) {
        lockedIndexes.add(index);
      } else {
        portsToReorder.push(port);
      }
    });

    // Sort ONLY the unlocked ports
    portsToReorder.sort((a, b) => {
      return this.comparePorts(a, b);
    });

    // Insert locked ports back in their original places
    const newPorts: Port[] = [];
    let reorderedIndex = 0;
    for (let index = 0; index < originalPorts.length; index++) {
      if (lockedIndexes.has(index)) {
        newPorts.push(originalPorts[index]);
      } else {
        newPorts.push(portsToReorder[reorderedIndex]);
        reorderedIndex += 1;
      }
    }
    this.ports = newPorts;
  }
```

[achrafmohye]

Done :)

[aiAdrian]

I am not sure whether we have to migrate the new added locked to the data migration. Netzgrafik-Editor must ensure that elder version will get automatically updated to latest version !

Please dbl-check if required to migrate - if yes -> add it to :

static migrateNetzgrafikDto(netzgrafikDto: NetzgrafikDto) {

https://github.com/OpenRailAssociation/netzgrafik-editor-frontend/blob/main/src/app/utils/data-migration.ts#L50

[achrafmohye]

I am not sure whether we have to migrate the new added locked to the data migration. Netzgrafik-Editor must ensure that elder version will get automatically updated to latest version !

Please dbl-check if required to migrate - if yes -> add it to :

static migrateNetzgrafikDto(netzgrafikDto: NetzgrafikDto) {

https://github.com/OpenRailAssociation/netzgrafik-editor-frontend/blob/main/src/app/utils/data-migration.ts#L50

Hi @aiAdrian,
I added locked = false in the Port constructor to handle the case where older Netzgrafik DTOs don’t include the locked field