tidev / gittio

Search & Install all Titanium Modules and Alloy Widgets on GitHub
http://gitt.io
Other
24 stars 19 forks source link

installing alloy widget causes TypeError #45

Closed manumaticx closed 10 years ago

manumaticx commented 10 years ago
Manuels-iMac:uww-app manu$ gittio install nl.fokkezb.infiniteScroll

/usr/local/lib/node_modules/gittio/node_modules/timodules/lib/timodules.js:22
              platform: curr.$.platform,
                              ^
TypeError: Cannot read property 'platform' of undefined
    at /usr/local/lib/node_modules/gittio/node_modules/timodules/lib/timodules.js:22:31
    at Array.forEach (native)
    at /usr/local/lib/node_modules/gittio/node_modules/timodules/lib/timodules.js:19:26
    at /usr/local/lib/node_modules/gittio/node_modules/timodules/node_modules/tiapp/lib/tiapp.js:19:15
    at Parser.<anonymous> (/usr/local/lib/node_modules/gittio/node_modules/timodules/node_modules/xml2js/lib/xml2js.js:255:20)
    at Parser.EventEmitter.emit (events.js:95:17)
    at Object.saxParser.onclosetag (/usr/local/lib/node_modules/gittio/node_modules/timodules/node_modules/xml2js/lib/xml2js.js:225:24)
    at emit (/usr/local/lib/node_modules/gittio/node_modules/timodules/node_modules/xml2js/node_modules/sax/lib/sax.js:615:33)
    at emitNode (/usr/local/lib/node_modules/gittio/node_modules/timodules/node_modules/xml2js/node_modules/sax/lib/sax.js:620:3)
    at closeTag (/usr/local/lib/node_modules/gittio/node_modules/timodules/node_modules/xml2js/node_modules/sax/lib/sax.js:861:5)
    at Object.write (/usr/local/lib/node_modules/gittio/node_modules/timodules/node_modules/xml2js/node_modules/sax/lib/sax.js:1293:29)
FokkeZB commented 10 years ago

Looks like an error in a dependency. Will look into it ASAP. Thx

manumaticx commented 10 years ago

I get the same when running gittio install in this project. I just tried in another project with success! Let me know if you need more info!

FokkeZB commented 10 years ago

Could you attach both projects' tiapp.xml files?

FokkeZB commented 10 years ago

I think in the project that doesn't work something is not right with the part between <modules> and </modules. Would be very interested to see it so the timodules dependency by David can be fixed for that.

FokkeZB commented 10 years ago

I think you have a <module>name</module> without platform or version attribute there.

FokkeZB commented 10 years ago

Closing as not our bug

manumaticx commented 10 years ago

These projects are on my offices desktop. I will attach the tiapp.xml files tomorrow.

dbankier commented 10 years ago

@FokkeZB, merged the PR and published to NPM. Update the dependency to 0.1.0

manumaticx commented 10 years ago

@FokkeZB You were right. Here are my module declarations from the tiapp.xml:

    <modules>
        <module platform="iphone">ti.storekit</module>
        <module platform="android">net.iamyellow.gcmjs</module>
        <module platform="android">ti.inappbilling</module>
        <module>ti.map</module>
    </modules>

ti.map caused the error. My fault. I've added the platform attribute and it worked out.