Bug #1744

MacroManager allows to import invalid macro file

Added by Zbigniew Rebacz almost 7 years ago. Updated almost 7 years ago.

Status:ApprovedStart date:06/16/2017
Priority:HighDue date:
Assignee:Zbigniew Rebacz% Done:

0%

Category:IDESpent time:-
Target version:Release 2017.2

Description

We need to fix two issues
- Importing .usc file should be valid - we cannot import file with controls definitions or another text file with changed extension.
- Even if the file is wrong it should be displayed in MacroManager - then user could delete it from them. So just display the file name and allow to open it and eventually fix.

The priority is high due to high impact on MacroManager usage. Thanks for reporting Mirek!

History

#1 Updated by Abdelghani Omari almost 7 years ago

Zbigniew Rebacz wrote:

We need to fix two issues
- Importing .usc file should be valid - we cannot import file with controls definitions or another text file with changed extension.
- Even if the file is wrong it should be displayed in MacroManager - then user could delete it from them. So just display the file name and allow to open it and eventually fix.

please give me a test case or scenario that describe the bug, the errorinouse behavior, and the wanted one.

now, a file is displayed only if it started with valid macro.

#2 Updated by Abdelghani Omari almost 7 years ago

i have changed a bit, please check.

#3 Updated by Zbigniew Rebacz almost 7 years ago

We want to display that wrong formated files, but the entry will not displayed the macro list. The main reason of that is to allow user remove the unwanted files on TheIDE level.

So for example if I do:
- cd UppLocal
- touch test.usc
- the file should be displayed on MacroManager level

The second scneario:
- I want to import test.usc (which is empty file or usc file with layout components)
- When you click on import button - the macro file should be validate - if it is wrong the error prompt should occurs and file should not be copied to UppLocal.

Now, is it much clear for you?

#4 Updated by Zbigniew Rebacz almost 7 years ago

  • Status changed from New to Approved
  • Assignee changed from Abdelghani Omari to Zbigniew Rebacz

Also available in: Atom PDF