Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Toolset - Drag and Drop of a DataType to the FB with validation error changes the values of namespace, name and version of the functionblock to null #655

Closed
deepagithub opened this issue Jun 27, 2017 · 2 comments

Comments

@deepagithub
Copy link
Contributor

Steps:

  1. Create a datatype e.g.
entity TightenResult {
	  optional cellId as int
	  optional channelId as int 
}
  1. Create a function block e.g.
functionblock Magnetometer {
	operations {
                getval() returns TightenResult
	}
}
  1. The function block has a validation error that the referenced model has to be imported
  2. Drag and Drop the entity to the function block

Expected Result:
The datatype should be included in the function block. The function block should have a statement "using.." referencing the data type.

Actual Result:
The Functionblock's namespace, name and version becomes null.
This is related to #622

@deepagithub deepagithub changed the title Drag and Drop of a DataType to the Function Block with Validation Error changes the values of namespace, name and version of the functionblock to null Toolset - Drag and Drop of a DataType to the FB with validation error changes the values of namespace, name and version of the functionblock to null Jun 27, 2017
@aedelmann aedelmann added the bug label Jun 28, 2017
@aedelmann aedelmann added this to the 0.10.0-M1 milestone Jun 28, 2017
@erlemantos
Copy link
Contributor

I can't reproduce this on Neon.2, Neon.3 and the latest Oxygen R with the latest nightly build

@erlemantos
Copy link
Contributor

I've been able to reproduce this, but only when the return type is fully qualified, not when it's just the 'name'. Anyhow, there are workarounds for this so I think this is not that critical.

@aedelmann aedelmann removed this from the 0.10.0-M1 milestone Jul 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants