-
Notifications
You must be signed in to change notification settings - Fork 329
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
"com.synconset.imagepicker" incorrectly adding to package.json dependencies #71
Comments
any update on this? |
How can solve this issue? |
I found a workaround by removing the single line |
@2010abinas It's solved for me |
@Bleno ,can you share the soluation of this problem? |
@qw1210p Was the same workaround commented by @2010abinas .
It worked for me, but I think not be the best solution. |
When i run ionic package build android got this error:
These problems are related? imagepicker plugin doesn't install on ionic cloud build. |
Every time ionic does an android build it parses the config.xml and adds all the plugins that it finds to the packages.json dependency list.
In this case, that's the wrong behavior because com.synconset.imagepicker is not a npm plugin. Then when you do an npm update it fails because this isn't a valid plugin.
The work around is to remove com.sysconset.imagepicker from the package.json every time you do a build and try to not check it in to your VCS system. Annoying. This doesn't seem like a "bug" on the ionic side, but rather a poorly thought out feature to keep the config.xml and packages.json in sync. This is also currently a massive gaping security vulnerability because anyone can, right now, register the com.synconset.imagepicker name on the npm registry and put whatever code they want on there. |
Hmmm not working for me. Any command such as:
Will fail because of trying to inject it:
If this is really needed, please make sure the package name is OK, or remove the dep. |
@EddyVerbruggen please any help with this? Thanks |
@djabif I'd be happy to use a PR but I don't have the time to do much more atm. |
in Ionic2 if you are using any existing project and you do |
Same issue faced after removing android platform try to re add it again and found this error `Installing "com.synconset.imagepicker" for android Error during processing of action! Attempting to revert... Failed to install 'com.synconset.imagepicker': Error: Uh oh! (node:89774) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Uh oh!
if any help ?! |
Hi There!! I have the same issue : Android project created with cordova-android@7.0.0 Android Studio project detected Installing "com.synconset.imagepicker" for android Error during processing of action! Attempting to revert... Failed to install 'com.synconset.imagepicker': Error: Uh oh! (node:24970) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Uh oh! |
Same issue with commands : Seems to be a folders and files organization issue, Cordova is searching in /platforms/android/AndroidManifest.xml, but the is there /platforms/android/app/src/main/AndroidManifest.xml I tried to update the paths everywhere I found it with no success. Is there a link with angular/schematics ? Using yarn here and at $ yarn install, I've got some warnings on schematics dependencies... |
@fedemoreno & @benpoucareformance Is there any fix for these issues??? |
@azeemse Yes, the only way I found is just about android package version, you need to downgrade at android 6.3.0. Here are my notes on this subject (you don't need to do it for ios if you don't have any problem): Check versions (see /config.xml or command "$ ionic info"):
Steps:
Run prepare command:
Reinstall platforms:
-- config.xml
-- package.json
It's a big waste of time, for sure. |
@benpoucareformance Thanks for sharing this. I was thinking of not downgrading the android version but it seems no other fix available right now. Even the Cordova-Android version 7.1.0 has also been launched but it still contains the same issue. |
@azeemse Yes it is, I tried to updated all paths in plugin script, but there was others issues in packages hooks. Better would too have another image picker plugin... Don't know if it exists... |
@benpoucareformance I have tried searching any other image picker plugin but couldn't get any thing latest. I am finally able to run my app on my Android device. :) |
Actually, it's not a Cordova version issue but a telerik image picker plugin issue I guess, they aren't update it to the new folder tree of Cordova Android version |
If so then at least Ionic should update their documentation with some verified image picker. They are still using the Telerik image picker in their docs. |
Hi, i'm having the same issue too.
But everytime i try to build or run in device the ionic tries to install the annoying com.synconset.imagepicker package back again. 👎 Does anyone have got the solution? |
Well, i've just noticed that if we want to completely remove the com.synconset.imagepicker from our projetct, we also need to remove it reference from plugins/fetch.json file. This way, cordova does not try to install it back in every build. |
Hi guys, follow the infos here, no need to downgrade cordova. It works on 7.1.0 |
These issues are fixed in pr 114 |
@Sjerd thanks! Part of 2.2.0 (now on npm). |
Hi , Ionic Framework : @ionic/angular 5.3.3 Cordova: Cordova CLI : 9.0.0 (cordova-lib@9.0.1) cordova.cmd plugin ls |
When using this plugin with Ionic (as per http://ionicframework.com/docs/native/image-picker/) it's breaking.
After running
ionic cordova plugin add cordova-plugin-telerik-imagepicker
andnpm install --save @ionic-native/image-picker
everything seems fine.But when you then go to use
ionic cordova run
orionic cordova build
a reference to"com.synconset.imagepicker"
is inserted into the package.json dependencies list. This is not a valid node module name, so it fails the next timenpm install
is run.The text was updated successfully, but these errors were encountered: