Skip to content

Fix site_title ReferenceError caused by third-party plugin conflicts#332

Merged
superdav42 merged 1 commit intomainfrom
fix-checkout-conflict
Jan 29, 2026
Merged

Fix site_title ReferenceError caused by third-party plugin conflicts#332
superdav42 merged 1 commit intomainfrom
fix-checkout-conflict

Conversation

@superdav42
Copy link
Collaborator

@superdav42 superdav42 commented Jan 29, 2026 •
edited by coderabbitai bot
Loading

Summary

  • Initializes site_title in the Vue initial_data object and PHP checkout variables so it's always defined, even when third-party plugins (e.g., JetGridBuilder by Crocoblock) interfere with page rendering
  • Previously site_title was only set via a v-init directive on the HTML form field, making it fragile to rendering interference
  • site_url was already properly initialized; this brings site_title to parity

Test plan

  • Install and activate JetGridBuilder (Crocoblock) alongside Ultimate Multisite
  • Navigate to the new site creation page (e.g., /newsite)
  • Select a template and proceed to the ?step=create step
  • Verify no ReferenceError: site_title is not defined in the browser console
  • Verify site creation completes successfully

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features

    • Site title is now accessible and available during the checkout process

  • Chores

    • Updated translation file metadata

✏️ Tip: You can customize this high-level summary in your review settings.

@superdav42 @claude
Fix site_title ReferenceError caused by third-party plugin conflicts
3973766 
Initialize site_title in Vue initial_data so it's always defined,
even if a plugin like JetGridBuilder prevents the form field from
rendering. Previously site_title was only set via v-init directive
on the HTML element, making it fragile to rendering interference.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 29, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

A new site_title field is added to the checkout form. The backend PHP class passes site_title to the frontend, where it's integrated into the Vue instance data structure for use in the checkout context.

Changes

Cohort / File(s) Summary
Checkout Site Title Addition
assets/js/checkout.js, inc/checkout/class-checkout.php		 Added site_title field to checkout variables passed from backend to frontend Vue instance, enabling access to the site title in the checkout form context.
Translation Metadata
lang/ultimate-multisite.pot		 Updated POT-Creation-Date timestamp to reflect latest generation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A title now flows through the checkout's design,
From backend to Vue, a connection so fine,
With site_title in hand, the form shines so bright,
The rabbit hops onward—all working just right! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary change: adding site_title initialization to prevent a ReferenceError caused by third-party plugin conflicts, which aligns with the changeset modifications across checkout.js, class-checkout.php, and the POT file.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Jan 29, 2026

🔨 Build Complete - Ready for Testing!

📦 Download Build Artifact (Recommended)

Download the zip build, upload to WordPress and test:

🌐 Test in WordPress Playground (Very Experimental)

Click the link below to instantly test this PR in your browser - no installation needed!
Playground support for multisite is very limitied, hopefully it will get better in the future.

🚀 Launch in Playground

Login credentials: admin / password

@superdav42 superdav42 merged commit c6d398b into main Jan 29, 2026
12 checks passed
@superdav42 superdav42 deleted the fix-checkout-conflict branch January 29, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@superdav42