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

Database table civicrm_api_call: change type of column 'reply' #54

Open
arnalyse opened this issue Dec 3, 2021 · 2 comments
Open

Database table civicrm_api_call: change type of column 'reply' #54

arnalyse opened this issue Dec 3, 2021 · 2 comments

Comments

@arnalyse
Copy link

arnalyse commented Dec 3, 2021

Hi there 👋

We ran in into an issue where the response of a Civi API call couldn't be stored in the civicrm_api_call table's reply column, because it was too long.

The php error was:

mysqli_sql_exception: Data too long for column 'reply' at row 1 in mysqli_stmt->execute() 
(Line 184 in /vendor/civimrf/cmrf_abstract_core/CMRF/PersistenceLayer/SQLPersistingCallFactory.php).

We were able to fix this by setting the column type to longtext, and would like to discuss if this would make sense in general as well.

@jensschuppe jensschuppe added this to the 2.1.0 milestone Dec 3, 2021
@jensschuppe
Copy link
Collaborator

Sounds like a good idea to me - do you mind providing a PR for this, @arnalyse?

@arnalyse
Copy link
Author

Yep. Will do, but may take a while. Time before Christmas is rare… 😆 🎅 🎁

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

No branches or pull requests

2 participants