Bug 724473 (BRMS-418) - Cannot switch to Basic view when a fact is declared in the Advanced view of the package configuration
Summary: Cannot switch to Basic view when a fact is declared in the Advanced view of t...
Keywords:
Status: VERIFIED
Alias: BRMS-418
Product: JBoss Enterprise BRMS Platform 5
Classification: JBoss
Component: BRM (Guvnor)
Version: 5.1.0.ER2
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: BRMS 5.3.0.GA
Assignee: manstis
QA Contact: Jiri Locker
URL: http://jira.jboss.org/jira/browse/BRM...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-10-27 14:42 UTC by Jiri Locker
Modified: 2021-04-09 12:48 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
When using the advanced view of the BRMS web user interface if the user declared a fact and attempted to switch back to the basic view, the following error message was displayed, "There are errors in this view. Can not switch to guided mode until the errors are resolved". The error message has been updated to provide more information and now reads, "Defining declared types in this view is not allowed. Please remove all declared types from the view."
Clone Of:
Environment:
Last Closed: 2010-11-08 15:57:57 UTC
Type: Bug


Attachments (Terms of Use)
PackageConfigError.png (190.13 KB, image/png)
2010-10-27 14:43 UTC, Jiri Locker
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker BRMS-418 0 Major Closed Cannot switch to Basic view when a fact is declared in the Advanced view of the package configuration 2012-10-24 23:46:19 UTC

Description Jiri Locker 2010-10-27 14:42:02 UTC
securitylevel_name: Public

Open a blank package, switch to Advanced view and declare a simple fact:

declare Fact
field1: String
field2: String
end

When you try to switch back to Basic view, you get a message "There are errors in this view. Can not switch to guided mode until the errors are resolved" which doesn't exactly describe the problem ,if there's any (I don't think so). However, the declared fact is imported to the fact model of the package and can be used to build rules in Guided editor.

Comment 1 Jiri Locker 2010-10-27 14:43:47 UTC
Attachment: Added: PackageConfigError.png


Comment 2 Tihomir Surdilovic 2010-11-08 15:46:10 UTC
Link: Added: This issue depends GUVNOR-1095


Comment 3 Tihomir Surdilovic 2010-11-08 15:50:49 UTC
You are really only supposed to be able to declare imports and globals in the advanced view. Currently you can also declare facts, rules, functions which is not supposed to :) The fact that you created actually is declared and visible, but the code shows the error window which is not descriptive at all. Added more descriptive error messages that tell users that they should not be declaring functions/rules/declared types here but should use the corresponding guided editor features for it.



Comment 4 Tihomir Surdilovic 2010-11-08 15:57:57 UTC
Done, more appropriate error messages are now shown to user.

Comment 5 Jiri Locker 2010-11-23 13:05:22 UTC
Now I understand it is deprecated declaring facts through the Advanced view. The error message looks OK.

But, now it *is* possible to switch back to Basic view (even if there is a fact declared in the Advanced view). Hitting 'Save and validate configuration' instead of 'Basic view' button will do the task. Please include removing this inconsistency to resolve the issue.

Comment 6 Anne-Louise Tangring 2010-11-23 20:14:31 UTC
This is not a blocker for BRMS 5.1.0, moving to future.

Comment 7 Tihomir Surdilovic 2010-11-24 03:36:56 UTC
Jiri, I thought that was the plan just to warn users that they should not declare facts/rules/etc in that view and should use the appropriate guided editor features. 
From the description it was not clear what was requested to implement, but I believe a warning is better than a hard "no" because of users who may already be doing this and depend on it in their packages.



Comment 8 lcarlon 2011-08-22 04:39:36 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Will document in release notes as a known issue, or resolved issue depending on resolution.

Comment 9 Jiri Locker 2011-09-23 11:29:35 UTC
Tiho, I agree with your explanation. The alert messages do describe the problem well now. I think there is noting more to be done on this issue.

Comment 10 lcarlon 2011-09-26 00:09:40 UTC
Deleted Technical Notes Contents.

Old Contents:
Will document in release notes as a known issue, or resolved issue depending on resolution.

Comment 11 lcarlon 2011-09-26 00:36:00 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
When using the advanced view of the BRMS web user interface if the user declares a Fact and attempts to switch back to the basic view, the following error message was displayed, "There are errors in this view. Can not switch to guided mode until the errors are resolved". The error message has been updated to provide more information and now reads, "Defining declared types in this view is not allowed. Please remove all declared types from the view."

Comment 12 lcarlon 2011-10-05 05:29:07 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1 +1 @@
-When using the advanced view of the BRMS web user interface if the user declares a Fact and attempts to switch back to the basic view, the following error message was displayed, "There are errors in this view. Can not switch to guided mode until the errors are resolved". The error message has been updated to provide more information and now reads, "Defining declared types in this view is not allowed. Please remove all declared types from the view."+When using the advanced view of the BRMS web user interface if the user declared a fact and attempted to switch back to the basic view, the following error message was displayed, "There are errors in this view. Can not switch to guided mode until the errors are resolved". The error message has been updated to provide more information and now reads, "Defining declared types in this view is not allowed. Please remove all declared types from the view."

Comment 14 lcarlon 2012-04-02 01:47:04 UTC
I removed the needs release note flag as this was documented as a resolved issue in the 5.2 release notes.


Note You need to log in before you can comment on or make changes to this bug.