Merge pull request #215 from mashmatrix/fix-lookup-scope-selection

stomita · web-flow · commit 3b4f85891b3a · 2017-04-14T14:25:27.000+09:00
Fix lookup component's scope selection is not working
diff --git a/.storybook/__storyshots__/Lookup.shot b/.storybook/__storyshots__/Lookup.shot
@@ -10900,10 +10900,10 @@ exports[`Uncontrolled with Multi Scope`] = `
                         type="button">
                         <svg
                           aria-hidden={true}
-                          className="slds-icon slds-icon--x-small slds-icon-standard-account"
+                          className="slds-icon slds-icon--x-small slds-icon-standard-opportunity"
                           style={undefined}>
                           <use
-                            xlinkHref="/assets/icons/standard-sprite/svg/symbols.svg#account" />
+                            xlinkHref="/assets/icons/standard-sprite/svg/symbols.svg#opportunity" />
                         </svg>
                         <svg
                           aria-hidden={true}
diff --git a/src/scripts/Lookup.js b/src/scripts/Lookup.js
@@ -227,7 +227,7 @@ export class LookupSearch extends Component {
     );
   }
 
-  renderScopeSelector({ scopes, target, disabled }) {
+  renderScopeSelector({ scopes, targetScope: target, disabled }) {
     let targetScope = scopes[0] || {};
     for (const scope of scopes) {
       if (scope.value === target) {

Original file line number	Diff line number	Diff line change
`@@ -227,7 +227,7 @@ export class LookupSearch extends Component {`
`227`	`227`	`);`
`228`	`228`	`}`
`229`	`229`
`230`		`- renderScopeSelector({ scopes, target, disabled }) {`
	`230`	`+ renderScopeSelector({ scopes, targetScope: target, disabled }) {`
`231`	`231`	`let targetScope = scopes[0] \|\| {};`
`232`	`232`	`for (const scope of scopes) {`
`233`	`233`	`if (scope.value === target) {`