sync fixes
[app.jsdoc] / JSDOC / ScopeNamer.js
index 17d061c..97d1118 100644 (file)
@@ -7,7 +7,7 @@ Symbol = imports.Symbol.Symbol;
 
 /**
  * @class ScopeNamer
- * @extends Collapse
+ * @extends JSDOC.Collapse
  * @scope JSDOC
  * The point of this class is to iterate through the Collapsed tree
  * and add the property 'scopedName' to the tokens.
@@ -89,6 +89,7 @@ ScopeNamer = XObject.define(
         Symbol.srcFile = this.filename;
         this.createJSDOC = true;
         this.global = '_global_';
+        this.locals = this.locals || [];
        // console.dump(ar);
         
     }, 
@@ -104,7 +105,7 @@ ScopeNamer = XObject.define(
         debugCall : function(str)  {
             
             
-            //if (this.filename.match(/Scope\.js/)) return print(str);
+           // if (this.filename.match(/Date\.js/)) return print(str);
             return;
         }, 
         
@@ -116,7 +117,7 @@ ScopeNamer = XObject.define(
                 this.statements = this.collapse(this.tokens);
                 //print(JSON.stringify(this.s, null,4));
             }
-            
+            //if (this.filename.match(/Roo\.js/)) print(JSON.stringify(this.statements, null,4));
             //this.globalScope(this.statements);
             this.debugCall("build Symbols");
             //print (this.statements);
@@ -136,24 +137,28 @@ ScopeNamer = XObject.define(
         
         walkStatements: function(scope, statements)
         {
-            this.debugCall("walkStatements :" + scope ) ;            
+            this.debugCall("walkStatements :" + scope + '@' + statements[0][0].line) ;            
             var _this = this;
             var res = false;
             var isGlobal = scope == this.global;
            
+            
+           
             statements.some(function(st ) {
                 // handle change of scope..
                 if (isGlobal &&
                         st[0].jsdoc &&
                         st[0].jsdoc.getTag('scope').length
                    ) {
-                    //print(st[0].jsdoc.getTag('scope'));
+                    
+                    
                     scope = st[0].jsdoc.getTag('scope');
                     // might be an array..
                     if (typeof(scope) != 'String') {
                         scope = scope[0].desc;
                         
                     }
+                    print("SCOPE CHANGE TO:" + scope);
                     _this.global = scope;
                 }
                 
@@ -174,8 +179,9 @@ ScopeNamer = XObject.define(
              
             var name;
             var sn;
+            var token = false; 
             
-            var isGlobal = scope == this.global;
+            var isGlobal = scope ==   this.global;
             
             
             
@@ -195,6 +201,11 @@ ScopeNamer = XObject.define(
             
             while (null != (token = this.next())) {
                 
+                
+                // var xx =
+                 
+                
+                
                 //'function' 
                 //walkFunction(scope, name , args,  stmts  )
                 //
@@ -204,6 +215,9 @@ ScopeNamer = XObject.define(
                         name = isGlobal ? this.lookTok(1).data : '';
                         
                         this.walkFunctionDef(scope, name, this.lookTok(2).args, this.lookTok(3).items, token);
+                        this.next();
+                        this.next();
+                        this.next();
                         continue;
                     }
                      //a = function() { } << scope might be a  $this$={a}
@@ -229,6 +243,8 @@ ScopeNamer = XObject.define(
                         
                         
                         this.walkFunctionDef(scope, name, this.lookTok(1).args, this.lookTok(2).items, this.lookTok(-2));
+                        this.next();
+                        this.next();
                         continue;
                     }
                     
@@ -254,6 +270,8 @@ ScopeNamer = XObject.define(
                         jsdoc = jsdoc || this.lookTok(-2).jsdoc;
                     }
                     this.walkCall(scope, assign, token.data, this.lookTok(1).items, jsdoc);
+                    this.next();
+                        
                     continue;
                 }
                 
@@ -273,25 +291,40 @@ ScopeNamer = XObject.define(
                     // using @type {string} for example.
                     // FIXME - they should be registered as a property...
                     if (jd && jd.jsdoc && jd.jsdoc.getTag('type').length) {
-                        
                          
-                        name = false;
+                        //name = false;
+                        //continue;
+                        // wipe out the children..
+                        token.props = false;
                     }
-                    
-                     
-                        
+                    /*
+                    print(JSON.stringify({
+                        name : name,
+                        isGlobal : isGlobal,
+                        isVar : isVar
+                    }, null,4));
+                    */  
                         
                     if (!isGlobal) {
                         // not global.
                         if (isVar) {
+                            if (name && this.locals.indexOf(name) < 0) {
+                                this.locals.push(name);
+                            }
                             name = false;
+                            continue;
                         }
+                        if (this.locals.indexOf(name) > -1 ) {
+                            continue;
+                        }
+                        
+                        
                         if (name && !name.match(/^this\./)) {
                             name = false;
                         }
                         if (name && name.match(/^this\./) ) { 
                             // see if scope ends in prototype..
-                            print("SCOPE:" + JSON.stringify(scope));
+                            //print("q" + JSON.stringify(scope));
                             if (
                                 (scope.split('.').pop() == 'prototype') &&
                                 (name.split('.').length == 2)
@@ -311,12 +344,105 @@ ScopeNamer = XObject.define(
                     continue;
                 }
                  
-                // this.xxxx = (with jsdoc...)
                 
+                 //  'assignment..'
+                 // it's done on the third element to enable object assignment to work above..
+                //   a = .....
+                if (this.lookTok(-1).data == '=' && this.lookTok(-2).is('NAME')) {
+                    
+                    
+                    
+                    // could be var x = ..
+                    var jd = this.lookTok(-2).jsdoc ? this.lookTok(-2) : this.lookTok(-3);
+                    
+                    var isVar = this.lookTok(-3).name == 'VAR';
+                    
+                    // only register names of objects if 
+                    var name = this.lookTok(-2).data;
+                    
+                    // some a = {} can be hidden
+                    // using @type {string} for example.
+                    // FIXME - they should be registered as a property...
+                    if (jd && jd.jsdoc && jd.jsdoc.getTag('type').length) {
+                         
+                        //name = false;
+                        //continue;
+                        // wipe out the children..
+                        token.props = false;
+                    }
+                    /*
+                    print(JSON.stringify({
+                        name : name,
+                        isGlobal : isGlobal,
+                        isVar : isVar
+                    }, null,4));
+                    */
+                    
+                    // skip imports.
+                    if (token.data.match(/imports\./)) {
+                        name = false;
+                    }
+                    
+                        
+                    if (!isGlobal) {
+                        // not global.
+                        if (isVar) {
+                            if (name && this.locals.indexOf(name) < 0) {
+                                this.locals.push(name);
+                            }
+                            name = false;
+                            continue;
+                        }
+                        if (this.locals.indexOf(name) > -1 ) {
+                            continue;
+                        }
+                        
+                        if (name && !name.match(/^this\./)) {
+                            name = false; 
+                        }
+                        
+                        
+                        if (name && name.match(/^this\./) ) { 
+                            // see if scope ends in prototype..
+                            //print("SCOPE:" + JSON.stringify(scope));
+                            if (
+                                (scope.split('.').pop() == 'prototype') &&
+                                (name.split('.').length == 2)
+                            ){
+                                name = name.split('.').pop();
+                            } else {
+                                name = false;
+                            }
+                            
+                        }   
+                        
+                    } else {
+                        // should not happen...
+                        if (name && name.match(/^this\./)) {
+                            name = false; 
+                        }
+                    }
+                    
+
+                    if (name) {
+                        var symbol = new Symbol( scope +'.'+ name , false , "PROPERTY" ,  jd.jsdoc);
+                        symbol._token = this.lookTok(-2);
+                           
+                        this.addSymbol(symbol, jd.jsdoc);
+                        continue;
+                    }
+                    
+                    
+                     
+                }
                 
+                // TODO:
+                // this.xxxx = (with jsdoc...)
+
+                      
                 
                  
-                // standard flow....
+                // standard flow... - ignore opening {.
                 if (token.data == '{') { 
                     sn = new ScopeNamer(this);
                     //print("GOT { - walkings statuements;}");
@@ -324,10 +450,30 @@ ScopeNamer = XObject.define(
                         continue; // object..!?!?!? = ignore ???
                         print(JSON.stringify(token,null,4));
                     }
+                     
+                    
+                    //print ("GOT { = running walk statemengs on children?} @" + this.cursor);
                     sn.walkStatements(scope, token.items);
                     continue;
                 }
+                  if (this.filename.match(/Roo\.js/)   && token.data == '(')  {
+                    //print(JSON.stringify(token,null,4));
+                      
+                  }
+                // ( function() { 
+                if ((token.data == '(') && (token.items[0][0].data == 'function')) {
+                    sn = new ScopeNamer(this);
+                    sn.walkStatements('_private_', token.items[0][2].items);
+                    continue;
+                    
+                }
+                
+                
+                
             }
+            
+             this.debugCall("walkStatement (END):" + this.global +'/' +scope + '@' + this.tokens[0].line );
+           
         },
         
         walkFunctionDef : function (inscope, name, args, stmts, jsdocTok)
@@ -340,8 +486,9 @@ ScopeNamer = XObject.define(
                 this.addSymbol(symbol, jsdocTok.jsdoc);
             }
             var sn = new ScopeNamer(this);
-            sn.walkStatements(scope, stmts);
             
+            sn.walkStatements(scope, stmts);
+            this.debugCall("wallkFuncDef:  (END)" + inscope + '#' + name + '@' + this.look(0).line );
         },            
         
         
@@ -362,13 +509,22 @@ ScopeNamer = XObject.define(
                 // or... x = somecall(BASE, { ... object..})
                 case 'XObject.extend':
                 case 'Roo.apply':
+                case 'Roo.applyIf':
                     //print(JSON.stringify(items,null,4));
                     // SHOULD WE ADD SCOPE HERE???
+                    
+                    var arg0 = items[0][0].data;
+                    
                     var topscope = scope.split('.').pop();
                     
-                    scope = (topscope == items[0][0].data) ?
-                        items[0][0].data :
-                        scope + '.' + items[0][0].data;
+                    scope = (topscope == arg0) ? arg0 : scope + '.' + arg0;
+                    
+                    // see if arg0 is defined has a symbol..
+                    if (this.haveSymbol(arg0)) {
+                        scope = arg0;
+                    }
+                    
+                    
                     // 2nd arg is a object def
                     if (items[1][0].data != '{') {
                         return;
@@ -386,7 +542,8 @@ ScopeNamer = XObject.define(
                     
                     //print(JSON.stringify(items,null,4));
                     sn.walkObject(scope , false, items[1][0].props, jsdocTok );
-                    
+                    this.debugCall("walkCall(END): " + inscope  +':' + assign + '@' + this.look(0).line + ' ' + callname );
+           
                     return;
                 
                 
@@ -407,13 +564,16 @@ ScopeNamer = XObject.define(
                     
                     
                     sn.walkObject(scope + '.prototype', false, items[2][0].props );
+                    this.debugCall("walkCall(END): " + inscope  +':' + assign + '@' + this.look(0).line + ' ' + callname );
+           
                     return;
                 
                 
                 
             }
             
-            
+            this.debugCall("walkCall(END): " + inscope  +':' + assign + '@' + this.look(0).line + ' ' + callname );
+           
              
         },
                             
@@ -432,7 +592,13 @@ ScopeNamer = XObject.define(
             if (!items) {
                 return;
             }
-            print(typeof(items));
+            // our other scenario is where something is flagged as private, we will not investigate further
+            // we do add it however..
+            if (jsdocTok && jsdocTok.jsdoc && jsdocTok.jsdoc.getTag('private').length) {
+                return;
+            }
+            
+            //print(typeof(items));
             this.debugCall("wallkObject : " + scope + '#' + name + '@' + items[Object.keys(items)[0]].key.line);
             for( var k in items) {
                 var key = items[k].key;
@@ -516,9 +682,11 @@ ScopeNamer = XObject.define(
             //if (this.conf.treatUnderscoredAsPrivate && symbol.name.match(/[.#-]_[^.#-]+$/)) {
             //    symbol.isPrivate = true;
             //}
-            
+            if (ScopeNamer.builder.treatUnderscoredAsPrivate && symbol.name.match(/[.#-]_[^.#-]+$/)) {
+                symbol.isPrivate = true;
+            }
             // -p flag is required to document private things
-            if ((symbol.isInner || symbol.isPrivate) && !this.docPrivate) return;
+            //if ((symbol.isInner || symbol.isPrivate) && !this.docPrivate) return;
             
             // ignored things are not documented, this doesn't cascade
             if (symbol.isIgnored) return;
@@ -534,6 +702,13 @@ ScopeNamer = XObject.define(
             
             ScopeNamer.symbols.addSymbol(symbol);
         },
+        
+        haveSymbol: function(name)
+        {
+            return ScopeNamer.symbols.hasSymbol(name);
+             
+        },
+        
         addBuiltin : function(name) {
 
             var builtin = new Symbol(name, [], "CONSTRUCTOR", new imports.DocComment.DocComment(""));