Closed kranian closed 5 years ago
팝업 오픈하면서, 조회된 결과를 objects로 저장하는 사이드 이펙트로 인해 불필요한 정보가 조회되는 현상이 있습니다. 즉, 팝업 오픈 시 마다, objects가 변경되면서, Paper.js의 277라인의 (JSON.stringify(this.props.objects) !== JSON.stringify(nextProps.objects)) 부분이 동작하게 되면서, 상황에 따라 counter와 xlog 등의 데이터가 조회되네요.
조회된 데이터를 가지고 active 판단에 사용하고, 해당 선택된 object를 변경하는 부분을 다시 검토 부탁드립니다.
리뷰 감사 합니다. objects로 저장하는 사이드 이펙트가 있는줄 생각도 못했네요. 피드백 중심으로 다시 수정 해 볼게요
조회된 데이터를 가지고 active 판단 부 와 선택된 object를 변경하는 부분을 다시 재 검토해서 수정했습니다.
checked
object필터/선택창을 누르는 경우에 Object 갱신 하여 Active,InActive 여부를 확인 합니다. #162