135540 2006-10-12 22:18:55 +0000 JJ: Javascript Code Generation creates bad format methods (operations) 2007-02-12 15:05:35 +0000 1 1 2 Applications umbrello general unspecified unspecified Linux RESOLVED FIXED NOR normal --- 1 ediaz666 umbrello-devel 0 oldest_to_newest 476432 0 ediaz666 2006-10-12 22:18:56 +0000 Version: 1.5.5 (using KDE 3.5.5 "release 19.1" , openSUSE ) Compiler: Target: x86_64-suse-linux OS: Linux (x86_64) release 2.6.16.13-4-default Javascript Code Generator returns bad format operations: .... ClassA.prototype.method1 function( param1 ) .... it doesn't put the = sign between method's name and the function declaration 488370 1 okellogg 2006-11-20 21:13:20 +0000 Another Junior Job candidate. 508480 2 19624 compudop 2007-02-11 21:28:50 +0000 Created attachment 19624 patch for bug #135540 Hello, the bug was in file codegenerators/jswriter.cpp This was indeed a junior job. Have a nice day 508644 3 okellogg 2007-02-12 15:05:34 +0000 SVN commit 632848 by okellogg: writeOperations(): Aply attachment 19624 from Antoine Dopffer. Thanks Antoine for contributing. BUG:135540 M +4 -0 ChangeLog M +1 -1 umbrello/codegenerators/jswriter.cpp --- branches/KDE/3.5/kdesdk/umbrello/ChangeLog #632847:632848 @@ -1,3 +1,7 @@ +Version 1.5.7 +* Bugs fixed from http://bugs.kde.org: +* Javascript Code Generation creates bad format methods (135540) + Version 1.5.61 * Copy/paste of attribute or operation in list view within same class --- branches/KDE/3.5/kdesdk/umbrello/umbrello/codegenerators/jswriter.cpp #632847:632848 @@ -223,7 +223,7 @@ js << " */" << m_endl; }//end if : write method documentation - js << classname << ".prototype." << cleanName(op->getName()) << " function " << "("; + js << classname << ".prototype." << cleanName(op->getName()) << " = function " << "("; int i = atl.count(); int j=0; 19624 2007-02-11 21:28:50 +0000 2007-02-11 21:28:50 +0000 patch for bug #135540 135540.patch text/plain 646 compudop SW5kZXg6IHVtYnJlbGxvL3VtYnJlbGxvL2NvZGVnZW5lcmF0b3JzL2pzd3JpdGVyLmNwcAo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSB1bWJyZWxsby91bWJyZWxsby9jb2RlZ2VuZXJhdG9ycy9qc3dyaXRlci5jcHAJ KHLDqXZpc2lvbiA2MzI2MTApCisrKyB1bWJyZWxsby91bWJyZWxsby9jb2RlZ2VuZXJhdG9ycy9q c3dyaXRlci5jcHAJKGNvcGllIGRlIHRyYXZhaWwpCkBAIC0yMjMsNyArMjIzLDcgQEAgdm9pZCBK U1dyaXRlcjo6d3JpdGVPcGVyYXRpb25zKFFTdHJpbmcgYwogICAgICAgICAgICAganMgPDwgIiAq LyIgPDwgbV9lbmRsOwogICAgICAgICB9Ly9lbmQgaWYgOiB3cml0ZSBtZXRob2QgZG9jdW1lbnRh dGlvbgogCi0gICAgICAgIGpzIDw8IGNsYXNzbmFtZSA8PCAiLnByb3RvdHlwZS4iIDw8IGNsZWFu TmFtZShvcC0+Z2V0TmFtZSgpKSA8PCAiIGZ1bmN0aW9uICIgPDwgIigiOworICAgICAgICBqcyA8 PCBjbGFzc25hbWUgPDwgIi5wcm90b3R5cGUuIiA8PCBjbGVhbk5hbWUob3AtPmdldE5hbWUoKSkg PDwgIiA9IGZ1bmN0aW9uICIgPDwgIigiOwogCiAgICAgICAgIGludCBpID0gYXRsLmNvdW50KCk7 CiAgICAgICAgIGludCBqPTA7Cg==