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 unused variable #22545

Merged
merged 1 commit into from
Jan 17, 2022
Merged

Remove unused variable #22545

merged 1 commit into from
Jan 17, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Remove unused variable

This is over-written a couple of lines later in the template so
we don't need it & ergo don't need to assign it either

Before

image

After

poof

Technical Details

Note that the paramter is assigned in postProcess for the
receipt so this is just this usage

Comments

This is over-written a couple of lines later in the template so
we don't need it & ergo don't need to assign it either

Note that the paramter is assigned in postProcess for the
receipt so this is just this usage
@civibot
Copy link

civibot bot commented Jan 17, 2022

(Standard links)

@civibot civibot bot added the master label Jan 17, 2022
@seamuslee001 seamuslee001 merged commit 47b64be into civicrm:master Jan 17, 2022
@seamuslee001 seamuslee001 deleted the build branch January 17, 2022 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants