Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 724473 (BRMS-418)

Summary: Cannot switch to Basic view when a fact is declared in the Advanced view of the package configuration
Product: [JBoss] JBoss Enterprise BRMS Platform 5 Reporter: Jiri Locker <jlocker>
Component: BRM (Guvnor)Assignee: manstis
Status: CLOSED UPSTREAM QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 5.1.0.ER2   
Target Milestone: ---   
Target Release: BRMS 5.3.0.GA   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/BRMS-418
Whiteboard:
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."
Story Points: ---
Clone Of: Environment:
Last Closed: 2025-02-10 03:13:50 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
PackageConfigError.png none

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.

Comment 20 Red Hat Bugzilla 2025-02-10 03:13:50 UTC
This product has been discontinued or is no longer tracked in Red Hat Bugzilla.