[PEPPOL] Add PEPPOL SE localization: Swedish organisation number for endpoints under scheme 0007

[Franco111000]

What and why

Swedish PEPPOL BIS 3.0 documents fail at the access point because the supplier/customer electronic address (cbc:EndpointID) and the cac:PartyLegalEntity/cbc:CompanyID are emitted as the full 14-character Swedish VAT number (for example SE733078715601) under schemeID="0007". Scheme 0007 is SE:ORGNR, which per PEPPOL-COMMON-R049 must be exactly the 10-digit Swedish organisation number (7330787156).

Per review feedback, the fix is implemented as a new PEPPOL SE localization app (src/Apps/SE/PeppolSE), following the PEPPOL DE pattern, and the earlier changes to the shared W1 PEPPOL codeunit are fully reverted:

• PEPPOL 3.0 Format enum extension adds PEPPOL 3.0 - SE Sales and PEPPOL 3.0 - SE Service, binding the PEPPOL Party Info Provider to a new PEPPOL30 SE Party Info codeunit (standard W1 validation and iterator are reused).
• The SE party info provider delegates every method to the standard PEPPOL30 implementation and post-processes only four of them: the endpoint identifiers are reduced to the 10-digit organisation number when the emitted schemeID is 0007, and the legal entity identifiers when the party's Country/Region VAT Scheme is 0007. GLN setups (0088), Denmark (0184), and Swedish parties registered under EAS 9955 (SE:VAT, where the full VAT is correct) are untouched by construction. cac:PartyTaxScheme/cbc:CompanyID (BT-31/BT-48) keeps the full VAT number.
• New PEPPOL 3.0 Setup records default to the SE formats on insert, mirroring PEPPOL DE. Existing setups keep their configured format; an install/upgrade step re-defaulting existing SE tenants can be added if preferred.
• A PEPPOL SE Tests app covers the provider through the enum interface dispatch, plus a posted sales invoice XML export.

Note: for a blank bill-to country the country scheme falls back to the company's country, which is the same fallback the W1 implementation already uses to emit the endpoint schemeID, so the value stays consistent with the declared scheme.

Linked work

Fixes #7723

How I validated

• Unit tests through the SE format dispatch: supplier and customer EndpointID and PartyLegalEntity/CompanyID return 7330787156 under scheme 0007
• Regression test: an SE party under scheme 9955 keeps the full VAT (not stripped)
• Test: PartyTaxScheme/CompanyID keeps the full VAT
• Test: a newly created PEPPOL 3.0 Setup defaults to the SE formats
• End-to-end test: a PEPPOL BIS 3.0 sales invoice export asserts both endpoints and the preserved PartyTaxScheme
• Relied on PR CI for compilation and the full test run (this repo builds via AL-Go)

Risk and compatibility

• The shared W1 PEPPOL app is unchanged in this PR; every other country is byte-for-byte unaffected.
• The SE behaviour only activates when the SE formats are selected in PEPPOL 3.0 Setup (defaulted for new setups in the SE localization).
• No schema, table, or API changes.

[Franco111000]

Pushed b4b5c7604 addressing the review feedback and the failing build:

• Split the new Swedish organisation-number branches so each if starts its own line (the build was red on AA0018); this is purely stylistic and the logic is unchanged.
• Documented the digit-extraction step in GetSwedishOrgNo (per the inline suggestion).
• Merged the latest main.

Also replied inline on the silent-fallback point.

[github-actions]

Copilot PR Review

Iteration 3 · Outcome: completed

All sub-skills ran; al-privacy-review found no domain-matching worklist candidates (no-knowledge) and al-ui-review found no page/UI files in the diff (not-applicable). No sub-skill produced findings at or above the Medium/minor severity floor; the self-review cross-cutting pass produced no candidate clearing the precision bar.

Knowledge source: https://github.com/microsoft/BCQuality@822cae1b2771ac25f665f73369f69093bd4fd630

Orchestrator pre-filter (13 file(s) excluded)

• layer-disabled (knowledge) : 13 file(s)

_{Findings produced by the Copilot CLI agent against BCQuality at 822cae1b2771ac25f665f73369f69093bd4fd630. Reply 👎 on any inline comment to flag false positives.}

[Franco111000]

Pushed 6fd2805c06 fixing the failing test ExportXml_PEPPOL_BIS3_SalesInvoice_SEOrgNo: validating the changed Country/Region Code on Company Information clears City and Post Code, so the PEPPOL company checks failed with "City must have a value". The test helper now restores both after switching the company to SE. Test-only change; the app code is untouched.

[Groenbech96]

I think the proper design is to create swedish localization, that implemetns the GetAccountingSupplierPartyInfo.
There are examples of this for Norway, BE, and other countries.

So should be fairly trivial change. Long term better design choice.

[Groenbech96]

See my comment.

[Franco111000]

@Groenbech96 Reworked as suggested in b6c2f8f55c: the W1 changes are fully reverted and the fix now lives in a new PEPPOL SE localization app (src/Apps/SE/PeppolSE), following the PEPPOL DE pattern.

• The SE party info provider delegates to the standard PEPPOL30 implementation and only reduces the endpoint and legal entity identifiers to the 10-digit organisation number when they are emitted under scheme 0007, so GLN setups and Swedish parties registered under EAS 9955 (SE:VAT) are untouched, and PartyTaxScheme keeps the full VAT number.
• New PEPPOL 3.0 Setup records default to the SE formats on insert, mirroring PEPPOL DE. Existing setups keep their configured format; happy to add an install/upgrade step if you would prefer existing SE tenants to be re-defaulted.
• Tests moved to a new PEPPOL SE Tests app, exercising the provider through the format dispatch plus a posted invoice XML export.
• Both apps are wired into the Apps SE build project and build/projects.json; please double-check the internal-path details of the PEPPOL SE entries there, I mirrored PEPPOL DE and SE Core Tests.

The PR description is updated to match.

[Groenbech96]

Great. Maybe consider install codeunit?
To set the SE versions oon the setup table that lives in W1.

[Franco111000]

@Groenbech96 Thanks! Both points addressed:

• Install codeunit added in c6d2aadbde: at install it moves an existing PEPPOL 3.0 Setup to the SE formats, but only when the current value is the W1 default, so a deliberately configured format is not overridden. New setups keep being defaulted by the insert subscriber.
• The failing Verify App Changes check was the new apps missing from build/groups.json; registered PEPPOL SE and PEPPOL SE Tests in 2ddc2a6305 with the same group memberships as PEPPOL DE and PEPPOL Tests (the country settings generator now validates clean locally).