C11: the country list never received the configured research URL
Found by the bug-class sweep. RegionCountryMap is passed :research-url; the UupgsList beside it is not — zero research mentions in its entire tag. So a host setting profile-config.researchUrl got it honoured on the map popup's Full Profile button and silently ignored by every profile link in the country list, which fell back to a hardcoded doxa.life default. What makes it the sweep's most useful find is the shape rather than the line: two consumers of one config, only one wired, and the unwired one carrying a plausible hardcoded default. A missing binding that fell back to an empty string would have rendered a visibly broken link; this one renders a working link to somebody else's site. It is invisible in either file alone — the parent's tag list has to be read against the children's props. One line to fix. gates:pre 40/40.
This commit is contained in:
parent
741783b4a5
commit
c5f61f7f8b
1 changed files with 6 additions and 0 deletions
|
|
@ -1212,12 +1212,18 @@ useShadowStyles(
|
|||
|
||||
<!-- CountryDetail.vue:63-69 — <UupgsList :lock-country-code hide-see-all-link />, and
|
||||
nothing around it: their page puts no heading above this block. -->
|
||||
<!-- C11, found by the bug-class sweep: RegionCountryMap already received :research-url and
|
||||
this list did not, so a host configuring researchUrl got it honoured on the map popup's
|
||||
Full Profile link and silently ignored by every profile link here. One config, two
|
||||
surfaces, two answers — and the wrong one fell back to a hardcoded doxa.life default,
|
||||
so it looked like a working link rather than a missing binding. -->
|
||||
<UupgsList
|
||||
v-if="subject.kind === 'country' && selectedCountry"
|
||||
:lock-country-code="selectedCountry.code"
|
||||
:pray-base-url="prayBaseUrl"
|
||||
:language-code="languageCode"
|
||||
:images-base="assetsBase + '/images'"
|
||||
:research-url="researchUrl"
|
||||
/>
|
||||
</template>
|
||||
</main>
|
||||
|
|
|
|||
Loading…
Reference in a new issue