Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecated: Array and string offset access syntax with curly braces is deprecated in twingle.civix.php on line 246 #51

Closed
MarcMichalsky opened this issue Aug 31, 2021 · 7 comments
Labels
status:fixed The issue has been resolved (usually by committing/merging code)

Comments

@MarcMichalsky
Copy link
Contributor

Screenshot 2021-08-31 at 10-59-15 CiviCRM Home CiviCRM Sandbox on Drupal

This error occurs in the civicrm-buildkit-docker (PHP 7.4.22).

MarcMichalsky added a commit to MarcMichalsky/de.systopia.twingle that referenced this issue Aug 31, 2021
MarcMichalsky added a commit to MarcMichalsky/de.systopia.twingle that referenced this issue Aug 31, 2021
@MarcMichalsky
Copy link
Contributor Author

Looks like this is a problem in the code that was automatically generated by civix. I got the same issue in my extension. 😅

@bjendres
Copy link
Member

Yes, exactly, we just have to run civix again with the current version to regenerate the files.

@MarcMichalsky
Copy link
Contributor Author

MarcMichalsky commented Sep 1, 2021

Yes, exactly, we just have to run civix again with the current version to regenerate the files.

If this is is the preferred way to fix this issue, I will close my PR #52.

jensschuppe added a commit that referenced this issue Sep 6, 2021
@jensschuppe
Copy link
Collaborator

cadf895 regenerates the twingle.civix.php file with latest Civix. @MarcMichalsky Could you run a short test to see if that solves the issue and does not impose other problems (which it really shouldn't)? This will then replace your PR #52.

@MarcMichalsky
Copy link
Contributor Author

cadf895 regenerates the twingle.civix.php file with latest Civix. @MarcMichalsky Could you run a short test to see if that solves the issue and does not impose other problems (which it really shouldn't)? This will then replace your PR #52.

With commit cadf895 there are no more errors occurring in my buildkit environment. I was also able to do a successfully TwingleDonation.submit call.

Looks fine to me. Thanks!

@jensschuppe
Copy link
Collaborator

@MarcMichalsky Thanks, I'll close this, since the commit is on master already.

@jensschuppe jensschuppe added the status:fixed The issue has been resolved (usually by committing/merging code) label Nov 9, 2021
@jensschuppe
Copy link
Collaborator

Will be released with 1.3 and 1.2.2.

jensschuppe added a commit that referenced this issue Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:fixed The issue has been resolved (usually by committing/merging code)
Projects
None yet
Development

No branches or pull requests

3 participants