Skip to content

Issue #228766 fix: ccAvenue: Billing Details >> Name and phone value does not get autofilled#82

Open
vaibhavsTekdi wants to merge 1 commit intotechjoomla:j4xfrom
vaibhavsTekdi:ccAvenue
Open

Issue #228766 fix: ccAvenue: Billing Details >> Name and phone value does not get autofilled#82
vaibhavsTekdi wants to merge 1 commit intotechjoomla:j4xfrom
vaibhavsTekdi:ccAvenue

Conversation

@vaibhavsTekdi
Copy link
Copy Markdown

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

// Notify URL set for notify url parameter of payment gateway
$gatewaydata['notify_url'] = $vars->notify_url;
$gatewaydata['billing_name'] = $vars->userInfo['firstname'] . ' ' . $vars->userInfo['lastname'];
$gatewaydata['billing_name'] = $vars->user_firstname . ' ' . $vars->user_lastname;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vaibhavsTekdi $vars is the value we get from the extension we can add $vars->user_firstname and $vars->user_firstname from the extension itself.

If we change or remove $vars->userInfo['firstname'] it might break the backward compatibility and might stop working for other extensions

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants