Fix KoJsonTrader scanning incorrect directories
ClosedPublic

Authored by alvinhochun on Feb 13 2018, 3:50 PM.

Details

Summary

I actually found this a while ago when working on 871a70823e8ffa007d8b59e18122b5f5f173b6d4, when I was referring to KoJsonTrader to write the function findKritaPythonLibsPath I found that the code in KoJsonTrader lists a lot of redundant entries. It also doesn't skip . and .., thus creating a chance of scanning a wrong plugin directory outside the application.root.

This change fixes the issue and cleaned up the flow a little bit.

Diff Detail

Repository
R37 Krita
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
alvinhochun created this revision.Feb 13 2018, 3:50 PM
Restricted Application added a subscriber: woltherav. · View Herald TranscriptFeb 13 2018, 3:50 PM
alvinhochun requested review of this revision.Feb 13 2018, 3:50 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 1 2018, 12:34 PM
This revision was automatically updated to reflect the committed changes.