Skip to content
This repository has been archived by the owner on Apr 22, 2019. It is now read-only.

Feature/avs 421 74 expand customer attributes for customer code #101

Merged

Conversation

rsisco
Copy link

@rsisco rsisco commented Dec 28, 2017

AVS-421_74 - Expand Customer Code config setting to include custom customer attributes

Richard Sisco added 2 commits December 28, 2017 13:39
- Add custom customer attributes to customer code option list
- Refactor queue processor to use custom attribute codes when defined
  and selected
Copy link
Contributor

@erikhansen erikhansen left a comment

Choose a reason for hiding this comment

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

Overall, looks good. Just need to make two syntactical changes and then we're good.


// Retrieve all customer attributes
$customerAttributes = $this->customer->getAttributes();
foreach($customerAttributes as $attribute){
Copy link
Contributor

Choose a reason for hiding this comment

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

Add space after foreach

$label = $attribute->getDefaultFrontendLabel();
if (!is_null($label) && $attribute->getIsUserDefined()) {
// Only add custom attribute codes that have a frontend label value defined
$attributesArray[$attribute->getAttributeCode()] = array(
Copy link
Contributor

Choose a reason for hiding this comment

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

Switch from array() to []

@rsisco rsisco merged commit 1ac5106 into develop Dec 29, 2017
@rsisco rsisco deleted the feature/AVS-421_74-expand-customer-attributes-for-customer-code branch December 29, 2017 14:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants