[ omegat-Feature Requests-3450222 ] Improve confusing message 'The project cannot be verified'

[SourceForge.net <no...@so...>]

Feature Requests item #3450222, was opened at 2011-12-04 04:30
Message generated for change (Comment added) made by guido_leenders
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=520350&aid=3450222&group_id=68187

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: OmegaT Application
Group: None
>Status: Deleted
Resolution: Duplicate
Priority: 5
Private: No
Submitted By: Guido Leenders (guido_leenders)
Assigned to: Nobody/Anonymous (nobody)
Summary: Improve confusing message 'The project cannot be verified'

Initial Comment:
In Main: selectProjectConsoleMode this message is given when verifyProject does not return success: 'The project cannot be verified'.

In ProjectProperties.java:

    public boolean verifyProject() {
        // now see if these directories are where they're suposed to be
        File src = new File(getSourceRoot());
        File loc = new File(getTargetRoot());
        File gls = new File(getGlossaryRoot());
        File tmx = new File(getTMRoot());
        File dict = new File(getDictRoot());
        if (!dict.exists()) {
            if (getDictRoot().equals(projectRoot + OConsts.DEFAULT_DICT + File.separator)) {
                dict.mkdirs();
            }
        }

        if (src.exists() && loc.exists() && gls.exists() && tmx.exists())
            return true;

        return false;
    }

This works fine. However, it leaves the writer of the batch script puzzling with what was wrong. Is it the source directory? Is it the target directory? Glossary? TM? He does not know.

Suggestion: print a more meaningfull error such as
The project cannot be verified; the source directory h:\ws\p104\xxx does not exist (which is an invalid directory on UNIX).

The method verifyProject might return a concatenated list of problems it has found, which can be printed along the "Project cannot be verified".

Additionally, I would like to suggest printing errors to stderr instead of stdout.

----------------------------------------------------------------------

>Comment By: Guido Leenders (guido_leenders)
Date: 2012-05-20 16:09

Message:
Moved to feature requests and already implemented.

----------------------------------------------------------------------

Comment By: Didier Briel (didierbr)
Date: 2011-12-04 04:42

Message:
I have moved to Feature Requests.

As you write, it works fine, but it could be improved.

Didier

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=520350&aid=3450222&group_id=68187