lvlOverride logic adjustments

[anti-the-social]

Moi :)

Currently concrete numbering related to overwriting starting value is applied to one level only.

Sometimes users want to start next concrete numbering with other level, than first one, but if we keep "overrideLevel" logic as it, numbering would be continued. Here is how it looks like: (pref 1.2 meant to be pref 1.1 since its restarted)

In order to fix this behaviour we have to be able to generate similar xml to this:

And this is the reference to the place where its hardcoded:

docx/src/file/numbering/numbering.ts

Line 214 in 563e10f

overrideLevel:

Potentially we can just propogate override for all levels with number 1, if instance has changed...But that would not be flexible e.g. you would be able to start with 10.1.1 after reset, but wouldn't be possible with 10.1.10

The best way, I guess, is to give extensive control over which level has to be actually reseted e.g. [{lvl: 0, start: 1}, {lvl:1, start : 10}]

[dolanmiu]

Hello :)

Hmm yeah I see what you mean, may require significant change in API you reckon?

[anti-the-social]

Yep, it does sound to me as a quite a break changes.
instance at IConcreteNumberingOptions seem to be not enough to be "as flexible as possible"

Nevertheless, we have an option where instance would just set all levels to 1 and optional overrideLevel.start would modify first level only. That might stay compatible with old versions...

[dolanmiu]

Do you have time to make a PR on this?

[anti-the-social]

Which way we want to go?
I might do PR, but wouldn't promise it to be fast 😅
Lets see how critical that would be to the project I am working on.

[dolanmiu]

Sure thing

Maybe something like this could be cool:

const concreteNumbering = new ConcreteNumbering({
    numId: 0,
    abstractNumId: 1,
    reference: "1",
    instance: 0,
    overrideLevels: [{
        num: 1,
        start: 1,
    }, {
        num: 2,
        start: 1,
    }, {
        num: 3,
        start: 1,
    }],
});

So turn overrideLevel into overrideLevels breaking change? Then release 8.0.0

Also, what is your project/company? Mind if I add it to the README?

[dolanmiu]

Closing as fixed by #1823