-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
example doesn't work #73
Comments
@hansemannn can you please explain to him? Thanks |
Mhh, I didn't touch the Android naming, damn. "ti.collectionview" is easier to use (e.g. for Alloy views like |
and please check the examples in the example/ and android/example/ folders, they are different and crashes with alloy Thanks |
@m1ga do you want change the mod id to "ti.collectionview" in android and bump to version 2.0 ? |
sure: #76 |
Many thanks to both @hansemannn and @m1ga you guys rock! |
I'm trying android 2.0 version with your last example but it seems doesn't works. ` [ERROR] : TiApplication: java.lang.RuntimeException: Unable to start activity ComponentInfo{com.test.collection/org.appcelerator.titanium.TiActivity}: java.lang.NullPointerException: Attempt to invoke virtual method 'void de.marcelpociot.collectionview.CollectionViewTemplate$DataItem.setDefaultProperties(org.appcelerator.kroll.KrollDict)' on a null object reference |
@ottopic Thanks, seems the example was not updated to reflect the new mod. id name. Some widgets like: nl.fokkezb.pullToRefresh and nl.fokkezb.infiniteScroll may need add this "ti.collectionview" the array, like I did in the example -> 4062319#diff-48e9c12d1852e32be67c8c6b697b42b3R18 |
@nuno that's the problem. I have used your example https://github.com/nuno/TiCollectionView/blob/master/example/app.js that report correct name and android dist 2.0.0 (https://github.com/nuno/TiCollectionView/blob/master/dist/ti.collectionview-android-2.0.0.zip) . So, how I can solve it? |
@ottopic please make sure the name is correct on tiapp.xml or you have to give me more info. with the example fixed and with correct name on tiapp.xml should work. Please clear the project first before the next build ;) |
@nuno I can confirm the problem because I have created new project to test it. I add the module width Titanium interface than I use:
I can attach the project if necessary. Probably the problem is the scroll to refresh |
@nuno thank you. No problem for the bug, the goal is fix it. :) |
attach the project please |
I was running the example without any errors so I'm not sure if changing the |
@ottopic try it without nl.fokkezb.pullToRefresh |
right, I removed that widget, since it isn't part of the module (and if you use 6.2.2 you can use the build in Refresh SDK!) |
@AhmedMohamed2014 |
The Alloy version works fine. Looks like the vanilla versions makes problems |
Thanks @m1ga also with my last commit I changed all the names to |
@nuno thank you but my app is already build in Vanilla mod and I can't change to Alloy. I'll wait the fix. |
Are there news about this fix? Does someone try to solve it? |
The example doesn't work and crash with alloy example.
The real module name is 'de.marcelpociot.collectionview', so why you write 'ti.collectionview'
The text was updated successfully, but these errors were encountered: