Skip to content

Instantly share code, notes, and snippets.

@skyofdwarf
Created June 24, 2022 07:38
Show Gist options
  • Save skyofdwarf/348ea15460fe33e9008b56442982c9cd to your computer and use it in GitHub Desktop.
Save skyofdwarf/348ea15460fe33e9008b56442982c9cd to your computer and use it in GitHub Desktop.
YPImagePicker 버그

YPImagePicker 버그

참여한 프로젝트에서 사진첩으로 YPImagePicker를 사용하고 있었는데 iOS 14에 추가된 '사진 선택'권한 관련한 기능추가 후 테스트를 하다 만난 YPImagePicker 버그들을 간단히 정리해본다.

이슈는 크게 '사진 선택' 권한으로 사진첩 사용 중 권한사진을 추가/삭제 할 경우 오동작과, 인덱스 기반의 선택항목 관리로 인한 오동작 2가지이다.

대충 겪은 이슈들은

  • 앨범 A에서 마지막 사진 P 선택하고 사진 P가 있지만 사진 수가 더 적은 앨범 B로 이동 후, 다른 사진을 선택 후 재선택 하면 크래쉬
  • 멀티셀렉션 상태에서 사진을 1개만 선택후 진행하면 crop 미적용
  • 사진 선택 후, 권한을 제거해도 진행이 가능하며 계속하면 크래쉬
  • n번째 사진 A를 선택 후 crop 조정하고 다른 앨범 변경 후 n번째의 다른 사진 B를 선택 후 crop 조정하면 사진 A의 crop이 변경되는 문제
  • 권한사진 추가/삭제 시, 선택목록의 인덱스가 맞지 않아 발생하는 무언가(머였지?)

사진첩 사용 중 권한사진 추가/삭제로 인한 이슈들은 그렇다 쳐도, 멀티선택 관리를 인덱스 기반으로 하면서 발생하는 이슈들은 라이브러리의 신뢰도를 떨어트렸다. 앨범 변경하면서 멀티선택만 해봐도 확인이 됐는데...

작업중인 프로젝트는 YPImagePicker를 CocoaPods이 아닌 소스를 포함하고 UI를 수정하여 사용하고 있었기에 위 목록에 나열된 문제들 포함해 직접 수정을 했다.

인덱스 대신 모두 localIdentifier를 사용하도록 수정

겪은 이슈중 하나는 PR로도 올라와 있던데(fix changing albums crash #747) 나도 PR하고 싶은 맘이 생기더라. 하지만 원본소스에서 작업한게 아니라 원소스 받고 재작업도 해야하고 어차피 영어PR은 해본적도 없다는 걸림돌. 수정한 내역을 보려면 회사에서 해야하는데 그럴 시간이 넉넉하지는 않고 나중에 집에서 하려면 회사소스는 못 보니 겪은 이슈를 모두 정리라도 해놔야 수정을 할텐데(했던걸 또 하라고??)...

라이브러리도 이슈가 있는거에 실망하고(이건 뭐 오픈소스인데! 실망하지 말고 고쳐야지), 이 프로젝트도 QA를 했는데 문제를 발견 못 한거에 실망하고, PR할 용기가 없는 나에게 실망하고.

하지만 난 로또 1등이 될거야.

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