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

Remove defunct default return property #20139

Merged
merged 1 commit into from
May 15, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Remove defunct default return property

Before

I was unable to get anything returned in this field - I think it predates websites having
their own table & is unused. Note that the BAO_MessageTemplateTest classs
shows 'url' being populated in tokens but this not being populated

After

No-one else should have to think about this ever again

Technical Details

Comments

I was unable to get anything returned in this field - I think it predates websites having
their own table & is unused. Note that the BAO_MessageTemplateTest classs
shows 'url' being populated in tokens but this not being populated
@civibot
Copy link

civibot bot commented Apr 25, 2021

(Standard links)

@civibot civibot bot added the master label Apr 25, 2021
@eileenmcnaughton
Copy link
Contributor Author

@colemanw @demeritcowboy do one of you want to check my logic on this - I think it was an actual field once....

@demeritcowboy
Copy link
Contributor

I had started to look at this and got sidetracked. I can take a look.

@eileenmcnaughton
Copy link
Contributor Author

thanks @demeritcowboy - I tested by creating a contact with various websites attached & then doing a contact_get & confirming it stayed empty

@demeritcowboy demeritcowboy merged commit 71fa9fa into civicrm:master May 15, 2021
@eileenmcnaughton
Copy link
Contributor Author

Oh wow! you found it!

@eileenmcnaughton eileenmcnaughton deleted the home_url branch May 15, 2021 02:26
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