ParseError: syntax error, unexpected identifier "HEXADECIMAL_CHANNEL_LENGTH", line 31 of AlphaColor.php

Created on 5 September 2025, 13 days ago

Problem/Motivation

Updating ui_skins 1.1.0-alpha3 => 1.1.0-alpha4 throws error:

ParseError: syntax error, unexpected identifier "HEXADECIMAL_CHANNEL_LENGTH", expecting "=" in Composer\Autoload\includeFile() (line 31 of /web/modules/contrib/ui_skins/src/Element/AlphaColor.php

Steps to reproduce

Upgrade ui_skins 1.1.0-alpha3 => 1.1.0-alpha4: composer update

Then, attempt db update: vendor/bin/drush updb

Proposed resolution

Unsure, haven't finished reviewing the latest commit yet.

Remaining tasks

User interface changes

API changes

Data model changes

πŸ› Bug report
Status

Active

Version

1.1

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States paskainos

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @paskainos
  • πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·
  • πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·

    Which PHP version are you using?

    Starting 1.1.0-alpha4, ui_skins requires PHP 8.3, but it seems that updating the .info.yml did not put the requirement into the automatically generated composer.json by drupal.org facade.

  • πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·

    After discussion on infrastructure Slack channel, PHP requirement from .info.yml is not extracted, need to create a composer.json file.

    Then indicate in previous release the problem.

  • πŸ‡ΊπŸ‡ΈUnited States paskainos

    Thank you, @grimreaper for your speedy response.

    The site in question is running PHP 8.2 - mystery solved! Should've / could've anticipated that. My apologies for not including that crucial bit in (let alone testing for it prior to) my initial post.

    After discussion on infrastructure Slack channel, PHP requirement from .info.yml is not extracted, need to create a composer.json file.

    Then indicate in previous release the problem.

    I'm not quite sure what that means. I guess my question is; do you need me to do anything further at this point?

  • πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·

    Hello @paskainos,

    No worries, I will finish this.

    Should've / could've anticipated that.

    1: I should have (and will) update release note to mention this new requirement
    2: I didn't know that this .info.yml requirement was not taken automatically so I will also fix other contrib modules I maintain.
    3: the .info.yml change only prevents installation, so for websites with the module already enabled it is not handled

    So unfortunately, very little stuff you could have done no worries. Except looking at the whole diff but that's not something normally asked when doing updates ^^.

    So it will be ok for next releases, composer will prevent the update, but not for 1.1.0-alpha4 so I will put a big warning message on release note.

  • Merge request !26[#3545021] fix: Add composer.json β†’ (Merged) created by Grimreaper
  • πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·
  • Now that this issue is closed, please review the contribution record.

    As a contributor, attribute any organization helped you, or if you volunteered your own time.

    Maintainers, please credit people who helped resolve this issue.

Production build 0.71.5 2024