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

Clarify return on BAO_Activity::create #26022

Merged
merged 1 commit into from
Apr 7, 2023

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Clarify return on BAO_Activity::create

Before

The declared return types from BAO_Activity::create are CRM_Activity_BAO_Activity|null|object but it only ever returns a BAO_Activity

After

Declared return type corrected. Legacy handling for error object removed

Technical Details

The save function doesn't return an error object - although I believe it did before we made exceptions the consistent error handling method - so I removed that handling

Comments

@civibot
Copy link

civibot bot commented Apr 7, 2023

(Standard links)

@civibot civibot bot added the master label Apr 7, 2023
@colemanw colemanw merged commit 8322eeb into civicrm:master Apr 7, 2023
@colemanw colemanw deleted the clarify_return branch April 7, 2023 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants