JabRef fix issue 8133, 8198

1. JabRef
전북대
17학번 홍승준
1

2. 순서
- JabRef 소개
- 작업한 내용 - 이슈내용, 해결과정
1. Issue 8198 – switching issue
2. Issue 8133 – default setting issue
2

3. 개요
JabRef : Java를 활용한 참고 문헌 관리 소프트웨어
3

4. 4

5. 작업한 내용
5

6. 1. Issue 8198 – switching issue
6

7. 이슈 내용
7

8. 과정
LibraryPropertiesDialogViewModel.java
8

9. 과정
SaveOrderConfig에서 정의한 fromBooleans 메소드
9

10. 1차 해결
LibraryPropertiesDialogViewModel.java 에서 사용한 fromBoolean 메소드
SaveOrderConfig에서 정의한 fromBooleans 메소드
10

11. 1차 해결
11

12. 1차 해결
JabRefPreferences.java에 있는 fromBooleans 메서드 사용
12

13. 1차 해결
13

14. comment
해석:
당신의 커밋은 기존에 있던
‘EXPORT_IN_ORIGINAL_ORDER’라는 변수가
무시되도록 합니다.
새로운 설정 변수(preferences variable)를
만들지 말고 함수의 logic을 수정하세요.
fromBooleans 메 소 드 를 다 음 과 같 이
수정하시길 바랍니다.
첨언: default 값은 JabRefPreferences 파일에
있는 defaults-map에서 바꾸시면 됩니다.
14

15. 최종 해결
다시 변경 – LibraryPropertiesDialogViewModel.java
15

16. 최종 해결
다시 변경:
JabRefPreferences.java
16

17. 최종 해결
해석:
관리자님 제안대로 fromBooleans
메소드를 정의한다면...
EXPORT_IN_ORIGINAL_ORDER 의
값은 TRUE가 되야 해서 다음과 같이
바꾸려고 합니다.
이렇게 바꿔도 괜찮나요?
17

18. 최종 해결
18

19. 최종 해결
19

20. 어려웠던 점
코드 분석…
하지만 좋은 경험인 건 확실함
20

21. 2. Issue 8193 – default issue
21

22. 이슈 내용
22

23. 과정
23

24. 과정 LibraryPropertiesDialogViewModel.java 의 생성자
OrderType
ORIGINAL : keep original order
TABLE : Use current table sort order
SPECIFIED : Use specified sort order
24

25. 과정
Preferences::getExportSaveOrder
25

26. 해결
26

27. 해결
27
SaveOrderConfig.java 파일

28. 해결
cf. valueOf 메서드
OrderType.valueOf(“SPECIFIED”) ==> return “specified”
OrderType.valueOf(“ORIGINAL”) ==> return “original”
OrderType.valueOf(“TABLE”) ==> return “table”
28

29. 해결
29

30. comment
해석 :
toUpperCase는 맞는 거 같
습니다.
제 생각에는 valueof가
case insensitive 한 줄 알
았는데 말이죠.
Case insensitive: 대소문자에 민감하지 않는 특
징
30

31. 해결
31

32. 어려웠던 점
코드 분석도 분석이지만…
이슈에 대한 코멘트의 중요성
32
숲 속에서
잡초 찾아야 하는 기분

33. Merged!!!!
33

34. 끝
34