Petals Studio

EIP croquis: add a warning instead of an error for not set operations

Details

  • Type: Improvement Request Improvement Request
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 1.2.0
  • Fix Version/s: 1.2.0
  • Component/s: Petals Tools
  • Security Level: Public
  • Description:

    When an operation is not set in an EIP croquis, a warning should be set instead of an error.
    Indeed, for some components (like SOAP), setting the operation does not make sense.
    Same thing for the MEP.

  • Environment:
    All

Activity

Hide
mistria added a comment - Wed, 6 Jul 2011 - 19:12:16 +0200

Need review on EipChainTransactionalValidator#validateConnection

Show
mistria added a comment - Wed, 6 Jul 2011 - 19:12:16 +0200 Need review on EipChainTransactionalValidator#validateConnection
Hide
Vincent Zurczak added a comment - Thu, 7 Jul 2011 - 14:45:18 +0200

Commit # 2067
The operation, if set, must be a qualified name (=> error).
Fix the error message for warnings.

Show
Vincent Zurczak added a comment - Thu, 7 Jul 2011 - 14:45:18 +0200 Commit # 2067 The operation, if set, must be a qualified name (=> error). Fix the error message for warnings.
Hide
mistria added a comment - Tue, 19 Jul 2011 - 17:49:20 +0200

Why was this bug reopened?

Show
mistria added a comment - Tue, 19 Jul 2011 - 17:49:20 +0200 Why was this bug reopened?
Hide
Vincent Zurczak added a comment - Tue, 19 Jul 2011 - 18:00:56 +0200 - edited

Because in a quick test, I noticed error levels were switched.
What should be an error is a warning and what is an error should be a warning. I need to check it back, I noticed it on Sunday... Maybe it was not a good day.
Sorry about the lack of explanations.

Show
Vincent Zurczak added a comment - Tue, 19 Jul 2011 - 18:00:56 +0200 - edited Because in a quick test, I noticed error levels were switched. What should be an error is a warning and what is an error should be a warning. I need to check it back, I noticed it on Sunday... Maybe it was not a good day. Sorry about the lack of explanations.
Hide
Vincent Zurczak added a comment - Wed, 20 Jul 2011 - 21:37:57 +0200

Commit # 2099
No message switch in the validation of a connection. But there were still some wrongs things in the validation.
The properties of a connection were also updated, so that typing an operation name by hand works correctly.

Show
Vincent Zurczak added a comment - Wed, 20 Jul 2011 - 21:37:57 +0200 Commit # 2099 No message switch in the validation of a connection. But there were still some wrongs things in the validation. The properties of a connection were also updated, so that typing an operation name by hand works correctly.

People

Dates

  • Created:
    Mon, 4 Jul 2011 - 14:55:16 +0200
    Updated:
    Fri, 19 Aug 2011 - 10:11:05 +0200
    Resolved:
    Wed, 20 Jul 2011 - 21:37:57 +0200