Importer doesn't skip files with extensions removed from settings

• Type: Bug
• Status: Closed
• Resolution: Won't Fix
• Affects Version/s: 7.0.X, Master
• Fix Version/s: None
• Component/s: Knowledge Base
• Labels:
  • 7.1-could
  • QA-V
  • profiq
• Story Points: 1
• Fix Priority: 3

User deletes one of supported extensions from System settings, in our example .png. Then user imports .zip file (using importer) which contains .png files in images folder. Png files are uploaded anyway, even if they are not included in supported extensions list.
 Steps:

1. Navigate to Control Panel -> Configuration -> System Settings
2. Select Knowledge Base -> Knowledge Base Service
3. Remove one of Markdown Importer Image File Extensions, eg. .png, hit Save/Update
4. Navigate to Site -> Content -> Knowledge Base
5. Select  button and Import
6. Add .zip file including .png file, hit Save
7. View detail of article, .png file should be present

Expected result: Files that has removed extensions should not be present
Actual result: Files with extensions which were removed are still present

 
Reproducible:

• CentOS, MySQL 5.6, Tomcat 9.0.6, 7.0.x nightly Master git ID: 7effb9c2fa4f8fe2b25733bdad6ccbd7e84e8f54
• FF 59.0.2, Chrome 66.0.3359.117

Fix priority: S3 + L3 = 3