Opened 5 years ago

Closed 5 years ago

#30393 closed Cleanup/optimization (fixed)

Add validation of target argument in startapp command.

Reported by: oliver Owned by: oliver
Component: Core (Management commands) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: yes
Easy pickings: yes UI/UX: no

Description

When someone calls startapp command with a target argument, The app will overlay on the target directory.
If the target directory has invalid name, the app can not be imported.
So, I think it would be good to add validation on target directory's name like app name validation.

Change History (4)

comment:2 by Mariusz Felisiak, 5 years ago

Owner: changed from nobody to oliver
Summary: Add validation to target of startapp command → Add validation of target argument in startapp command.
Triage Stage: Unreviewed → Accepted
Type: Bug → Cleanup/optimization

​PR

comment:3 by Mariusz Felisiak, 5 years ago

Needs tests: set
Patch needs improvement: set

comment:4 by Mariusz Felisiak <felisiak.mariusz@…>, 5 years ago

Resolution: → fixed
Status: assigned → closed

In fc9566d:

Fixed #30393 -- Added validation of startapp's directory option.

Note: See TracTickets for help on using tickets.
Back to Top