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

Issue: Attachments on Information Fields #4730

Merged

Conversation

Projects
None yet
2 participants
@aydreeihn
Copy link
Contributor

commented Feb 13, 2019

This commit fixes an issue where we failed to upload attachments on Information Fields. In the to_config function, we were only taking the array_values of attachments which would return an array with only the file name ([0] => 'filename') instead of an array with the file id and file name ([123] => 'filename').

Now, when we pass the keepers to keepOnlyFileIds, we have a file id that can be passed to the upload function.

Issue: Attachments on Information Fields
This commit fixes an issue where we failed to upload attachments on Information Fields. In the to_config function, we were only taking the array_values of attachments which would return an array with only the file name ([0] => 'filename') instead of an array with the file id and file name ([123] => 'filename').

Now, when we pass the keepers to keepOnlyFileIds, we have a file id that can be passed to the upload function.

@protich protich merged commit a2aa8ae into osTicket:develop-next Feb 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.