Author: ben
Date: 2007-08-03 16:35:17 -0700 (Fri, 03 Aug 2007)
New Revision: 5931

Modified:
   openlaszlo/branches/wafflecone/WEB-INF/lps/lfc/services/LzCSSStyle.js
Log:
Change 20070803-ben-D by [EMAIL PROTECTED] on 2007-08-03 16:24:30 PDT
    in /Users/ben/src/svn/openlaszlo/branches/wafflecone
    for http://svn.openlaszlo.org/openlaszlo/branches/wafflecone

Summary: Better warning messages for nonexistent CSS property

New Features:

Bugs Fixed: LPP-4428 Make warning on CSS value is null more informative

Technical Reviewer: max 
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:
The warning if a developer tries to look up a CSS property which isn't
defined now lists the node, the node's id, and the node's name --
warnings only appear in debug mode. 

Release Notes:

Details:

Tests:
http://localhost:8080/wafflecone/test/style/resource/main.lzx
says

WARNING: No CSS value found for node LzView  id: gMissing  called gMissing for 
property name nostyledefined 




Modified: openlaszlo/branches/wafflecone/WEB-INF/lps/lfc/services/LzCSSStyle.js
===================================================================
--- openlaszlo/branches/wafflecone/WEB-INF/lps/lfc/services/LzCSSStyle.js       
2007-08-03 23:28:25 UTC (rev 5930)
+++ openlaszlo/branches/wafflecone/WEB-INF/lps/lfc/services/LzCSSStyle.js       
2007-08-03 23:35:17 UTC (rev 5931)
@@ -168,7 +168,11 @@
     if ($debug) {
         // Fix for LPP-3024: if we're in debug mode, warn when CSS lookup
         // results in a null or undefined value. [bshine 08.03.2007]
-        Debug.warn("There is no CSS property %s for node %s", pname, 
initialnode);
+        Debug.warn(
+            "No CSS value found for node %s called \"%s\" for property name 
%s",
+            initialnode, 
+            (initialnode.id ? initialnode.id : initialnode.name), 
+            pname);             
     }
 }
 


_______________________________________________
Laszlo-checkins mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-checkins

Reply via email to