gmane.comp.java.openlaszlo.devel

Syntactic sugar

André Bargull — 2008-12-03T20:27:58

If we had (native) setters in all runtimes, things would be much easier. I've you add an alias for "setAttribute", I'd like to remind you on the current limitation of the "setAttribute" inline compiler: So any new alias (let's say "set(..)") will be restricted to the same limitation. And there is another issue which comes to my mind: in my opinion, it's not only the longer method name ("setAttribute" instead of "setX"), but also the whole function call with attribute-name in quotes + value. Just as an idea, what about adding a new syntax? (I do know adding a non-ECMAScript syntax may be harmful..) For example to set the "x"-attribute to "10" on a view "v": set v.x = 10;

For Review: Change 20081203-Philip-0 Summary: Fix window_example to run in dhtml/swf9

Philip Romanik — 2008-12-03T16:25:31

Change 20081203-Philip-0 by Philip-DC>Philip< at >Philip-DC on 2008-12-03 11:10:00 EST in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: Fix window_example to run in dhtml/swf9 New Features: Bugs Fixed: LPP-7396 Technical Reviewer: jcrowley QA Reviewer: (pending) Doc Reviewer: (pending) Documentation: Release Notes: Details: When the color code was updated, support for a statement like "{iceblue1}" was removed. I also changed to a standard color name. Tests: window_example.lzx runs in swf8/swf9/dhtml. Files: M examples/components/window_example.lzx Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081203-Philip-0.tar

Chapter 33. Extending Classes

P T Withington — 2008-12-03T16:21:57

All the example code in this section seems to have lost its indentation [Chapter 33. Extending Classes](http://labs.openlaszlo.org/trunk-nightly/docs/developers/class-inheritance.html#settertag )

Chapter 20. Color

P T Withington — 2008-12-03T16:17:35

Do we really want stuff like this in the dguide? Is this something we can't handle with a quirk? [Chapter 20. Color](http://labs.openlaszlo.org/trunk-nightly/docs/developers/color.html#d0e46137 )

JIRA components: Documentation - RefGuide

Lou Iorio — 2008-12-03T12:57:36

Hi Amy, As far as I understand, doc bugs should be filed in one of two places: Documentation: all dguide issues Doc Tools: all reference guide issues Some people are filing bugs as Documentation - RefGuide, which, from their point of view makes perfect sense, but these are getting assigned to me, which doesn't make sense. These should be assigned to developers. Is there any legitimate reason for bugs to be filed as Documentation - RefGuide? Should Documentation - RefGuide just go away, and existing open bugs be moved to Doc Tools? If not, the default assignment should be to someone other than me. Thanks, Lou

For Review: Change 20081203-bargull-Srt Summary: check context's type

André Bargull — 2008-12-03T00:38:43

Change 20081203-bargull-Srt by bargull< at >dell--p4--2-53 on 2008-12-03 01:24:02 in /home/Admin/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: check context's type New Features: Bugs Fixed: LPP-7423 Technical Reviewer: hminsky QA Reviewer: (pending) Doc Reviewer: (pending) Documentation: Release Notes: Details: The context must be a LzDatapath, otherwise it doesn't make much sense to look at "__LZdepChildren". Tests: Files: M WEB-INF/lps/lfc/data/LzDatapath.lzs Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081203-bargull-Srt.tar

tag 'debug'cannot be used as a child of a library

Henry Minsky — 2008-12-02T15:10:10

I'm seeing this compiler warning in the test/lfc/data/alldata.lzx test datapointerdependencies.lzx:1:10: The tag 'debug' cannot be used as a child of library datapath.lzx:1:10: The tag 'debug' cannot be used as a child of library xpspace.lzx:1:10: The tag 'debug' cannot be used as a child of library I wonder when did this restriction get introduced? It seems reasonable actually, I don't think we don't want people putting the tag into libraries anyway, but I wonder what caused it to start warning? -- Henry Minsky Software Architect hminsky< at >laszlosystems.com

For Review: Change 20081202-lou-l Summary: remove lpsserver admin app links from internal index

Lou Iorio — 2008-12-02T15:06:32

Change 20081202-lou-l by lou< at >loumac.local on 2008-12-02 10:59:20 AST in /Users/lou/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: remove lps server admin app links from internal index Bugs Fixed: LPP-7390 Technical Reviewer: (pending) QA Reviewer: (pending) Doc Reviewer: (pending) Details: We don't ship this internal index, but there's no point in keeping these broken links around. Tests: visual verify Files: M index.jsp Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081202-lou-l.tar

For Review: Change 20081202-hqm-r Summary: fixlz.datapath sortorder schema type

Henry Minsky — 2008-12-02T14:47:11

Change 20081202-hqm-r by hqm< at >badtzmaru.home on 2008-12-02 09:45:15 EST in /Users/hqm/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: fix lz.datapath sortorder schema type New Features: Bugs Fixed: LPP-7422, LPP-7424 Technical Reviewer: andre QA Reviewer: promanik Doc Reviewer: (pending) Documentation: Release Notes: Details: + Type should be 'string', default to 'ascending' Tests: test case in LPP-7422 smokecheck swf9 Files: M WEB-INF/lps/lfc/data/LzDatapath.lzs Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081202-hqm-r.tar

For Review: Change 20081202-lou-C Summary: remove lpsserver admin app from dguide, laszlo in 10

Lou Iorio — 2008-12-02T13:40:40

Change 20081202-lou-C by lou< at >loumac.local on 2008-12-02 09:35:44 AST in /Users/lou/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: remove lps server admin app from dguide, laszlo in 10 Bugs Fixed: LPP-7390 Technical Reviewer: (pending) QA Reviewer: (pending) Doc Reviewer: (pending) Details: remove two brief mentions of the lps server admin app from the admin guide. No changes were needed for the dguide or laszlo in 10 Tests: visual verify Files: M docs/src/deployers/deployers-guide.dbk M docs/src/deployers/logging.dbk Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081202-lou-C.tar

For Review: Change 20081202-lou-1 Summary: dguide,Chapter 33. Extending Classes: add section 4. s

Lou Iorio — 2008-12-02T12:14:43

Change 20081202-lou-1 by lou< at >loumac.local on 2008-12-02 08:06:38 AST in /Users/lou/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: dguide, Chapter 33. Extending Classes: add section 4. s Bugs Fixed: LPP-5062 Technical Reviewer: (pending) QA Reviewer: (pending) Doc Reviewer: tucker Details: add Tucker's documentation section from LPP-2302 Tests: visual verify Files: M docs/src/developers/classes-powerprogramming.dbk Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081202-lou-1.tar

Out of memory

ono keiji — 2008-11-29T11:41:42

If this mail has been gotten multi-post, but my previous mails looks like undelivered. I got a out of memory error in building with rev11923. It was same as the report by Andre in July. Any advice? build-opt: init: [echo] Ant project:lps on Linux 2.6.26.6-49.fc8 i386 [echo] ant.home: /opt/workspace/openlaszlo/openlaszlo_trunk/lib/apache-ant-1.6.5 [echo] JRE: 1.5.0_14 (Sun Microsystems Inc.) [echo] JAVA_HOME: /usr/local/java/jdk1.5.0_14 [echo] LPS Version: 4.2.x Latest [echo] LPS_HOME: /opt/workspace/openlaszlo/openlaszlo_trunk/openlaszlo_trunk [echo] LZ_VENDOR_ROOT: /opt/workspace/openlaszlo/openlaszlo_trunk/vendor [echo] LZ_TOOLS_ROOT: /opt/workspace/openlaszlo/openlaszlo_trunk/tools/trunk [echo] LZ_INSTALLS_ROOT: /opt/workspace/openlaszlo/openlaszlo_trunk/lib [echo] server.home: /opt/workspace/openlaszlo/openlaszlo_trunk/lib/jakarta-tomcat-5.0.30 [echo] lps.branch: openlaszlo_trunk [echo] platform: is unix [echo] Skipping prefetch is true [echo] Skipping wrappertest is ${skip.wrappertest} [echo] Skipping doc is ${skip.doc} : [java] Stray error string from external compiler: [java] compiler output: [java] loading "/opt/workspace/openlaszlo/openlaszlo_trunk/openlaszlo_trunk/WEB-INF/frameworks/flex-config.xml" [java] SEVERE ERROR: error: Java heap space [java] SEVERE ERROR: java.lang.OutOfMemoryError: Java heap space [java] FAIL: compiler returned 1 [java] Done executing compiler [java] Intermediate file /tmp/lzswf9/lzgen3206/app.swc: does not exist [java] Exception compiling scriptfile: org.openlaszlo.sc.CompilerError: SEVERE ERROR: eror: Java heap space [java] SEVERE ERROR: java.lang.OutOfMemoryError: Java heap space [java] org.openlaszlo.sc.CompilerException: org.openlaszlo.sc.CompilerError: SEVERE ERROR: error: Java heap space [java] SEVERE ERROR: java.lang.OutOfMemoryError: Java heap space [java] at org.openlaszlo.sc.Compiler.compile(Compiler.java:397) [java] at org.openlaszlo.sc.lzsc.compile(lzsc.java:110) [java] at org.openlaszlo.sc.lzsc.compile(lzsc.java:325) [java] at org.openlaszlo.sc.Main.main(Main.java:10) [java] Compilation aborted. ono keiji ono< at >net8.co.jp

So long, and thanks for all the fish

Mamye Kratt — 2008-12-01T23:59:17

Today is my last day. It was a pleasure working with you and I wish Laszlo the best in the future. Mamye AIM: mamye42 EMAIL: mamye< at >comcast.net HOME: 781-391-8606

For Review: Change 20081130-ptw-v Summary: Revert initarg behavior

P T Withington — 2008-12-01T16:55:29

Change 20081130-ptw-v by ptw< at >dueling-banjos.home on 2008-11-30 12:01:16 EST in /Users/ptw/OpenLaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: Revert init arg behavior Bugs Fixed: Lpp-7378 LzTextSprite.setPattern warning in Laszlo 4.1.1, not in 4.0.12 LPP-7386 Proposal to revert behavior of LzNode.construct to 4.1 behavior Technical Reviewer: a.bargull< at >intensis.de (pending) QA Reviewer: max (pending) Details: Added a new internal API LzNode/__LZhasConstraint, which can be used in LzNode/construct to determine if an attribute will be constrained by LzNode/__LZapplyArgs. This replaces testing for LzInitExpr's in construct's arglist. LzNode/construct' initargs are now as they were in 4.1 -- only constant initial values appear in that list. Currently only LzText (and subclasses) use the __LZhasConstraint API, because they (apparently) need to know of their width/height will be constrained at construct time. No other LFC classes or component classes use this facility, so I have kept is as an LFC private API. LzTextSprite, LzInputTextSprite: These two sprite classes should not be using the __LZhasConstraint API, but they are probably going to be obsolete soon, so I just fixed them in the most expedient way. LzNode: Remove LzInitExprs from init args when calling construct, while preserving the ability of construct replacing/adding init args. LzText: Use new __LZhasConstraint API. LzText, LzInputText, LaszloView: Simplify construct method, now that args only contains constant values. LzReplicationManager: Clarifying comment hintedlayout: Fix construct arg defaulting bug noted in passing (similar to LPP-7376) Tests: smokecheck, ant test, test case from LPP-7386 Files: M WEB-INF/lps/lfc/kernel/swf/LzTextSprite.as M WEB-INF/lps/lfc/kernel/swf/LzInputTextSprite.as M WEB-INF/lps/lfc/kernel/swf9/LzTextSprite.as M WEB-INF/lps/lfc/core/LzNode.lzs M WEB-INF/lps/lfc/views/LzInputText.lzs M WEB-INF/lps/lfc/views/LzText.lzs M WEB-INF/lps/lfc/views/LaszloView.lzs M WEB-INF/lps/lfc/data/LzReplicationManager.lzs M demos/lzpixmobile/lib/hintedlayout.lzx Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081130-ptw-v.tar

For Review: Change 20081201-bargull-znZ Summary: add new style setter for "contextmenu"

André Bargull — 2008-12-01T16:08:10

Change 20081201-bargull-znZ by bargull< at >dell--p4--2-53 on 2008-12-01 15:57:19 in /home/Admin/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: add new style setter for "contextmenu" New Features: Bugs Fixed: LPP-7418, LPP-7408 (partial) Technical Reviewer: hminsky QA Reviewer: (pending) Doc Reviewer: (pending) Documentation: Release Notes: Details: SchemaBuilder: - introduce some new types (or the SchemaBuilder will fail to generate the schema) LzDatapointer: - add < at >type for context (the doc explains that this field points to a clonemanager, but there are actually more possibilities, I've enumerated them in a < at >devnote) LaszloView: - add < at >type for contextmenu - add "contextmenu"-setter - deprecate "setContextMenu(..)" - use "contextmenu" directly in "getContextMenu()" (deprecate this method, too?) - remove some legacy code LaszloCanvas: - update callers of "setContextMenu(..)" - add < at >type for "httpdataprovider" and "defaultdataprovider" - add description for "onmouseleave" (copied from LzGlobalMouse) LzContextMenu: - update callers of "setContextMenu(..)" - use the new style setters as the real setters - deprecate the old "setXXX(..)" methods Tests: Files: M WEB-INF/lps/lfc/views/LaszloView.lzs M WEB-INF/lps/lfc/views/LaszloCanvas.lzs M WEB-INF/lps/lfc/data/LzDatapointer.lzs M WEB-INF/lps/lfc/services/LzContextMenu.lzs M WEB-INF/lps/server/src/org/openlaszlo/js2doc/SchemaBuilder.java Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081201-bargull-znZ.tar

UPDATE: For Review: Change 20081128-bargull-Rhp Summary: UPDATE: schema build warnings

André Bargull — 2008-12-01T14:56:43

Change 20081128-bargull-Rhp by bargull< at >dell--p4--2-53 on 2008-11-28 22:53:41 in /home/Admin/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: UPDATE: schema build warnings New Features: Bugs Fixed: LPP-7408 (partial) Technical Reviewer: ptw QA Reviewer: (pending) Doc Reviewer: (pending) Documentation: Release Notes: Details: Class: Changed profile annotation, so it doesn't use a local (actually global) variable to shorten the assignment expression. LzNode: added < at >type LzFont: mark internal properties as < at >access private LzSelectionManager: rename argument to match java-doc LzMakeLoadSprite: rename argument (also a bugfix..) LzBrowserKernel: move doc into code (not really important here, it's a kernel class) LzDataset: rename arg in java-doc to match implementation LzReplicationManager: rename argument to match java-doc Tests: Files: M WEB-INF/lps/lfc/compiler/Class.lzs M WEB-INF/lps/lfc/core/LzNode.lzs M WEB-INF/lps/lfc/data/LzDataset.lzs M WEB-INF/lps/lfc/data/LzReplicationManager.lzs M WEB-INF/lps/lfc/helpers/LzFont.lzs M WEB-INF/lps/lfc/helpers/LzSelectionManager.lzs M WEB-INF/lps/lfc/kernel/swf/LzMakeLoadSprite.as M WEB-INF/lps/lfc/kernel/swf9/LzBrowserKernel.lzs Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081128-bargull-Rhp.tar

For Review: 20081201-laszlo-0 Summary: Datapointer.deleteNode() should check previous sibling if next sibling doesn't exist, Databinding example 23 exhibits odd behavior

J Crowley — 2008-12-01T10:57:23

Change 20081201-laszlo-0 by laszlo< at >T43-L3XEXMW on 2008-12-01 05:36:10 EST in /home/laszlo/src/svn/openlaszlo/trunk-grayfox for http://svn.openlaszlo.org/openlaszlo/trunk Summary: Datapointer.deleteNode() should check previous sibling if next sibling doesn't exist Databinding example 23 exhibits odd behavior New Features: Bugs Fixed: LPP-7416 - Datapointer.deleteNode() should check previous sibling if next sibling doesn't exist LPP-7220 - Databinding example 23 exhibits odd behavior Technical Reviewer: ptw QA Reviewer: andre.bargull< at >udo.edu Doc Reviewer: lou< at >louiorio.com Documentation: Release Notes: Details: LPP-7416 - Applied Andre's fix suggestion. LPP-7220 - Changed rerunxpath to false to get the example to function appropriately (with LPP-7416 fixed). Tests: Run databinding-$23.lzx. Click "Next" until you reach the end of the list. Click "Delete". You should now move to the previous node when the current one is deleted, if there is no next node. Compare to an unmodified version, where the data references an invalid node and then subsequently returns all nodes. Files: M WEB-INF/lps/lfc/data/LzDatapointer.lzs M docs/src/developers/programs/databinding-$23.lzx Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081201-laszlo-0.tar

For Review: Change 20081130-hqm-7 Summary: documentLzView.focusable as read/write

Henry Minsky — 2008-12-01T03:02:16

Change 20081130-hqm-7 by hqm< at >badtzmaru.home on 2008-11-30 21:03:47 EST in /Users/hqm/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: document LzView.focusable as read/write New Features: Bugs Fixed: LPP-6022 Technical Reviewer: max QA Reviewer: mamye Doc Reviewer: (pending) Documentation: Release Notes: Details: Tests: should say read/write in docs Files: M WEB-INF/lps/lfc/views/LaszloView.lzs Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081130-hqm-7.tar

For Review: Change 20081130-bargull-VjZ Summary: fix global color issues

André Bargull — 2008-11-30T14:41:59

Change 20081130-bargull-VjZ by bargull< at >dell--p4--2-53 on 2008-11-30 15:33:00 in /home/Admin/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: fix global color issues New Features: Bugs Fixed: LPP-7412 Technical Reviewer: (pending) QA Reviewer: (pending) Doc Reviewer: (pending) Documentation: Release Notes: Details: - use numeric constants instead of global values like "red" - changed "tint" and "fontcolor" type to "color" in stylishbutton.lzx Tests: Files: M lps/components/incubator/roundrectbutton.lzx M lps/components/incubator/stylishbutton.lzx M lps/components/incubator/test/opttree/treenode.lzx M lps/components/incubator/rich-text/linkdialog.lzx M lps/components/base/baselist.lzx Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081130-bargull-VjZ.tar

For Review: Change 20081130-bargull-a72 Summary:update alldata

André Bargull — 2008-11-30T12:54:32

Change 20081130-bargull-a72 by bargull< at >dell--p4--2-53 on 2008-11-30 13:34:17 in /home/Admin/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk Summary: update alldata New Features: Bugs Fixed: LPP-7411 Technical Reviewer: (pending) QA Reviewer: (pending) Doc Reviewer: (pending) Documentation: Release Notes: Details: Update deprecated methods to use setAttribute, in detail: - setDatapath(..) to setAttribute('datapath', ..) - setQueryType(..) to setAttribute('querytype', ..) - setPostBody(..) to setAttribute('postbody', ..) Tests: alldata still passes Files: M test/lfc/data/testrawpost-solo.lzx M test/lfc/data/testgetmethod.lzx M test/lfc/data/testsetheaders.lzx M test/lfc/data/dollarpath.lzx M test/lfc/data/datarepl.lzx M test/lfc/data/testheaderresponse.lzx M test/lfc/data/testgetmethodsolo.lzx M test/lfc/data/testsetpostbody.lzx M test/lfc/data/testdatapath.lzx M test/lfc/data/testrawpost.lzx M test/lfc/data/datapath.lzx M test/lfc/data/testpostmethod.lzx M test/lfc/data/testsetheaders-solo.lzx M test/lfc/data/repldone.lzx M test/lfc/data/testpostmethodsolo.lzx M test/lfc/data/datadollarpath.lzx M test/lfc/data/testsetpostbody-solo.lzx Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081130-bargull-a72.tar

setData() on LzDataText is deprecated?

André Bargull — 2008-11-30T12:23:59

Hi Henry, just a quick question: Do you know why "LzDataText#setData(..)" was deprecated, but none of: - "LzDataElementMixin#setAttrs(..)" - "LzDataElementMixin#setChildNodes(..)" - "LzDataElementMixin#setNodeName(..)" - "LzDataElementMixin#setOwnerDocument(..)" ? - André