Postcode Configuration Page shows an empty Address Provider when a provider is configured
Nobody has claimed this yet.
- Dominant language
- AL
- Stars
- 683
- Forks
- 459
- Avg merge
- 3d 26m
- Merged PRs (30d)
- 633
Description
Describe the issue
OnOpenPage of page 9143 "Postcode Configuration Page W1" (and the GB twin, page 10501 "Postcode Configuration Page") sets the display variable ServiceKeyText only in the record-missing and not-configured branches — both to Disabled. With a correctly configured provider neither branch runs, so the "Address Provider" field renders an empty value (shows as an underscore in the web client) instead of the selected provider. The stored key is already read into PrevValue; it is just never displayed.
Reproduced on BC 28.2 with a configured, working postcode provider. This was previously masked for IdealPostcodes because a provider bug (see the linked IdealPostcodes issue) kept IsConfigured() false, which forced the page into the Disabled branch.
Expected behavior
Reopening the Postcode Configuration Page shows the currently selected provider.
Proposed fix (implemented)
In both pages, replace the last statement of OnOpenPage:
// old
PrevValue := Rec.GetServiceKey();
// new
ServiceKeyText := Rec.GetServiceKey();
PrevValue := ServiceKeyText;
A PR with this change is ready on branch sergey-idealpostcodes:fix/postcode-configuration-page-show-selected-provider and will be opened once this issue is approved.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with OnOpenPage on pages 9143 and 10501, "Postcode Configuration Page W1" and "Postcode Configuration Page". Reopen each page with a configured provider and verify that the Address Provider field displays the selected provider rather than an empty value; the proposed fix is already implemented on a contributor branch.
Written by the indexing model from the issue text.
Assessment
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100