i encountered something I suspect could be common for some people rushing to use patterns on a clean install.

Situation

Upon attempting an import of a new pattern, I received a message that was cryptic: Destination folder doesn't exist.

I didn't know what it meant... so I modified patterns.module to tell me WHAT 'destination directory' didn't exist. After seeing the destination, I immediately knew I forgot to make my files writeable.

Suggested Solution

It might be nice to have an error message to both include the path that doesn't exist, and suggest a link to Status Report to ensure files folder is writeable.

That's what my patch does...

Typing this up, I realized I missed this error, in the dblog. perhaps the dblog has a better message to display on the form than in my patch. In any case, I feel an enhanced error message would improve usability.

CommentFileSizeAuthor
error-message.patch845 bytesmike stewart

Comments

vaish’s picture

Status: Needs review » Fixed

Hi Mike,

All usability improvements are very much welcomed.
I committed your patch after making some minor changes related to coding style.

Thanks,
Vaish

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.