BroadleafCommerce/QA#1508 - Don't replace MVEL expressions for the pricing context attributes since they are already Map<String, String>

Author: phillipuniverse

common/src/main/java/org/broadleafcommerce/common/rule/MvelHelper.java

@@ -208,14 +208,20 @@ protected static String modifyExpression(String rule, Map<String, Object> rulePa
         return modifiedExpression;
     }
 
+    /**
+     * Returns an array of attribute map field names that we need to do replacements for in
+     * {@link #modifyExpression(String, Map, ParserContext)}
+     */
     protected static String[] getRuleAttributeMaps() {
+        // intentionally left out pricing context getPricingContextAttributes because that's a Map<String, String>
         return new String[]{ "getProductAttributes",
             "getCategoryAttributesMap",
             "getSkuAttributes",
             "getOrderItemAttributes",
             "getCustomerAttributes",
-            "getPricingContextAttributes",
+            // Map<String, PageAttribute>
             "getAdditionalAttributes",
+            // Map<String, AdminUserAttribute>
             "getAdditionalFields"}; 
     }