I don't see any reason why. In any case, it's more understandable to check if a variable is None instead of "None". The issue happen when I import files from "Recently Used" folder. For some reason
self.frmAddFiles.get_current_folder() return None. It could be because "Recently Used" is not an actual folder and the library did not handle it probably. Then the value was assigned to preferences.Settings.app_state["import_folder"] without checking. This cause undesirable problem afterward since the check fail to invalidate the None value. The patch only remove the double quotes around "None", so no harm done I think. If needed, you could change it to if default_folder != None and default_folder != "None": It can't be be bad to do more check. Sincere -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/874906 Title: can't import files more than one time To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/openshot/+bug/874906/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
