The following errors show when using Field Hidden fields:

Deprecated: Array and string offset access syntax with curly braces is deprecated in field_hidden.admin.inc on line 198
Deprecated: Array and string offset access syntax with curly braces is deprecated in field_hidden.admin.inc on line 199

CommentFileSizeAuthor
#2 field_hidden-n3174496-2.patch1.66 KBDamienMcKenna

Issue fork field_hidden-3174496

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

    About issue forks
    Support from Acquia helps fund testing for Drupal Acquia logo

    Comments

    DamienMcKenna created an issue. See original summary.

    DamienMcKenna’s picture

    DamienMcKenna
    Location NH, USA
    CreditAttribution: DamienMcKenna at Mediacurrent commented
    Status: Active » Needs review
    FileSize
    field_hidden-n3174496-2.patch1.66 KB

    This should cover the problem.

    joelpittet’s picture

    joelpittet
    Primary language English
    Location Vancouver
    CreditAttribution: joelpittet as a volunteer and at The University of British Columbia commented
    Status: Needs review » Reviewed & tested by the community

    Thanks @DamienMcKenna, I was just about to write this patch :)

    aubjr_drupal’s picture

    aubjr_drupal CreditAttribution: aubjr_drupal commented

    @DamienMcKenna - Same as @joelpittet. Thanks. :)

    Paul Webster’s picture

    Paul Webster CreditAttribution: Paul Webster as a volunteer commented

    Major hosting provider Ionos (ex 1&1) plans to start charging an extra fee of 7GBP per month from January 2022 to sites that need to keep running PHP 7.3

    I had 3 modules that were generating PHP warnings because of 7.4 ... this is one of them.
    Have applied the patch from #2 and it resolves the warning with no obvious detrimental effect.

    Would be good to have it rolled into Drupal 7 (I am running Drupal 7.84 which is latest at time of writing this).

    sir_squall’s picture

    sir_squall CreditAttribution: sir_squall commented

    Do you know when this will be merged?

    jacobfriis’s picture

    jacobfriis CreditAttribution: jacobfriis commented

    It won't be merged. Just use the patch. The work involved in creating a new release is not justified by a deprecation issue in the D7 track (EOL in foreseeable future).

    DamienMcKenna’s picture

    DamienMcKenna
    Location NH, USA
    CreditAttribution: DamienMcKenna at Mediacurrent commented

    It's five minutes to commit the fix and make a new release?

    joelpittet’s picture

    joelpittet
    Primary language English
    Location Vancouver
    CreditAttribution: joelpittet as a volunteer and at The University of British Columbia commented

    @jacobfriis I’m willing to co-maintain D7 if that helps keep your focus on D9+ feel free to add me as a maintainer

    jacobfriis’s picture

    jacobfriis CreditAttribution: jacobfriis commented
    Status: Reviewed & tested by the community » Fixed

    Boy, oh boy. field_hidden 7.x-1.9 released ;-)

    joelpittet’s picture

    joelpittet
    Primary language English
    Location Vancouver
    CreditAttribution: joelpittet as a volunteer and at The University of British Columbia commented

    Thanks🚀 and if you want a hand my offer stands

    jacobfriis’s picture

    jacobfriis CreditAttribution: jacobfriis commented

    @joelpittet De nada ;-) I'll keep your offer in mind, if I get pressed for time, or (particularly) if something complex appears in D7 context.

    Apologies, everyone. I didn't take this serious, because I'm used to a (D9 + composer) context where adding|applying a patch is a very trivial one-off task.
    And because I think we should focus on D9 (which is easy to say if one doesn't work with D7 projects...).

    Anyway, if this is important for you guys, then I certainly must take it serious :-)

    Status: Fixed » Closed (fixed)

    Automatically closed - issue fixed for 2 weeks with no activity.