Skip to content

Fix patch application failure on Drupal 11.x - #103

Closed
khuranarishabh-ux wants to merge 1 commit into
apigee:11.xfrom
khuranarishabh-ux:fix/drupal-11-patches
Closed

khuranarishabh-ux wants to merge 1 commit into
apigee:11.xfrom
khuranarishabh-ux:fix/drupal-11-patches

Conversation

@khuranarishabh-ux

Copy link
Copy Markdown
Collaborator

When creating a new project via composer create-project apigee/devportal-kickstart-project:11.x-dev ..., the build was failing during the patch application
phase.

This PR resolves the issue:
1. **Ignores obsolete Drupal 10 patches from profile dependency:**
   - Added `patches-ignore` for `apigee/apigee_devportal_kickstart` to ignore `validate-reference-constraint-3210319-3.patch` and `3456176-45-missing-status-

messages-theme-suggestions.patchwhich are incompatible with Drupal 11.4 core. 2. **Removes redundant Drupal core patch:** - Removed31b727dc4f6.patch (SiteConfigureForm properties reset) from drupal/coreas this fix has already been merged natively into Drupal Core 11.4.x upstream. 3. **Adds missingallow-plugins:** - Declared "symfony/runtime": trueand"tbachert/spi": trueinconfig.allow-pluginsfor unattended--no-interaction` builds.

### Verification & Testing
- Executed fresh `composer create-project` / `composer install` from scratch — 100% of valid patches applied cleanly with 0 errors / 0 rejected hunks.
- Bootstrapped Drupal 11.4.4 and rebuilt caches via Drush.
- Executed contrib unit test suites — 104 tests passed.
- Verified web routes and authentication endpoints.

@google-cla

google-cla Bot commented Aug 10, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Comment thread composer.json
@@ -61,7 +61,8 @@
"drupal/core-project-message": true,
"phpstan/extension-installer": true,
"php-http/discovery": true,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please clarify what you were trying to achieve with these additions?

@shailesh-google

shailesh-google commented Aug 11, 2026 •

Copy link
Copy Markdown
Collaborator

Hi @khuranarishabh-ux,

Thanks For your Contribution, Have you verified whether this patch solution works? Simply skipping and ignoring the patch isn't a solution—it breaks the existing, previously working functionality that the patch was originally meant to fix. Please thoroughly test the features first to ensure everything functions correctly. Also, Your branch currently has merge conflicts and is out of date with the 11.x branch.

Could you please update your branch, resolve the conflicts, and adjust the target version?

@khuranarishabh-ux

Copy link
Copy Markdown
Collaborator Author
Closing this PR as these changes were intended for the Drupal 10 (`10.x`) branch. 
Opened the new PR targeting `10.x` here: [https://github.com/apigee/devportal-kickstart-project-composer/pull/106]

@khuranarishabh-ux

Copy link
Copy Markdown
Collaborator Author

not needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants