https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43887 Bug ID: 43887 Summary: Checkout warning block has empty id instead of circmessages Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: martin.renvoize@openfifth.co.uk QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle@bywatersolutions.com Depends on: 38714 Target Milestone: --- On the checkout page (circ/circulation.pl), the block that holds the "Cannot check out!" warning (and the on-site-checkouts-only attention message) should have id="circmessages". Since Bug 38714 it renders id="" instead. circulation.tt sets a variable div_id to 'circmessages' in each branch, but the div is then written as: <div id="[% circmessages | html %]" class="[% div_class | html %]"> i.e. it outputs the (undefined) variable circmessages instead of div_id. Consequences: - Custom IntranetUserCSS / IntranetUserJS that target #circmessages (it worked up to 24.11) no longer match. - The staff stylesheet rule for #circmessages (card styling: padding, bottom margin, white background) no longer applies to this block. With the id the block has 1rem padding, 1rem bottom margin and a white background; without it, 0.4rem padding, no margin and a transparent background. - The page emits an invalid empty id attribute. Affects 25.05 and later, including main. Test plan: 1. Have a patron who cannot check out (for example add a manual restriction to a patron). 2. Go to the checkout page for that patron. 3. Inspect the "Cannot check out!" block: the div has id="" and no card styling (no white background, tight padding). 4. Apply the patch and reload. 5. The div now has id="circmessages", and it has padding, bottom margin and a white background. 6. Optionally enable OnSiteCheckouts and OnSiteCheckoutsForce and confirm the attention variant also has id="circmessages". Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38714 [Bug 38714] Adjust templates for prettier -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.