-
Notifications
You must be signed in to change notification settings - Fork 9
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
Permet de faire un filepicker #17
Conversation
\o/ ça m'a l'air pas mal toussa. Je vais lire ça a tête reposée puis tester un peu. En lisant en diagonale, je vois que ça ne doit pas enregistrer les paramètre de l'utilisateur. C'est voulu ? |
C'est juste que je sais pas encore comment faire. ça fait 2 ans que j'ai Le 18 février 2016 à 11:19, firm1 notifications@github.com a écrit :
|
En fait moi la question que je me taraude c'est surtout : "ou est-ce que c'est plus simple de stocker notre fichier de configuration ?". Car la question se pose aussi sur la sauvegarde de la session des membres ( feature qu'a demandé @GerardPaligot ). |
ça c'est dans le AppData/Roaming Le 18 février 2016 à 12:11, firm1 notifications@github.com a écrit :
|
et sur linux dans un dotfile Le 18 février 2016 à 12:22, francois dambrine artragis@gmail.com a écrit :
|
Et sous OS X ? |
Je sais pas, je suis pas un pigeon. (mais troll à part, je sais vraiment Le 18/02/2016 13:32, firm1 a écrit :
|
C'est pour ça que je cherche un truc qui soit multiplateforme. Je n'arrive pas à trouver, je n'ai pas envie d'avoir des if windows, else if linux, else if ... |
Quand je lance ta PR, j'ai une erreur au run. java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at com.sun.javafx.application.LauncherImpl.launchApplicationWithArgs(LauncherImpl.java:389)
at com.sun.javafx.application.LauncherImpl.launchApplication(LauncherImpl.java:328)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at sun.launcher.LauncherHelper$FXHelper.main(LauncherHelper.java:767)
Caused by: java.lang.RuntimeException: Unable to construct Application instance: class com.zestedesavoir.zestwriter.MainApp
at com.sun.javafx.application.LauncherImpl.launchApplication1(LauncherImpl.java:907)
at com.sun.javafx.application.LauncherImpl.lambda$launchApplication$156(LauncherImpl.java:182)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.reflect.InvocationTargetException
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:422)
at com.sun.javafx.application.LauncherImpl.lambda$launchApplication1$162(LauncherImpl.java:819)
at com.sun.javafx.application.PlatformImpl.lambda$runAndWait$176(PlatformImpl.java:326)
at com.sun.javafx.application.PlatformImpl.lambda$null$174(PlatformImpl.java:295)
at java.security.AccessController.doPrivileged(Native Method)
at com.sun.javafx.application.PlatformImpl.lambda$runLater$175(PlatformImpl.java:294)
at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:95)
at com.sun.glass.ui.gtk.GtkApplication._runLoop(Native Method)
at com.sun.glass.ui.gtk.GtkApplication.lambda$null$50(GtkApplication.java:139)
... 1 more
Caused by: java.lang.NullPointerException
at java.io.File.<init>(File.java:277)
at com.zestedesavoir.zestwriter.utils.LocalDirectorySaver.<init>(LocalDirectorySaver.java:22)
at com.zestedesavoir.zestwriter.utils.FilePickerDirectorySaver.<init>(FilePickerDirectorySaver.java:20)
at com.zestedesavoir.zestwriter.utils.LocalDirectoryFactory.<init>(LocalDirectoryFactory.java:24)
at com.zestedesavoir.zestwriter.utils.ZdsHttp.<init>(ZdsHttp.java:191)
at com.zestedesavoir.zestwriter.MainApp.<init>(MainApp.java:41)
... 13 more
Exception running application com.zestedesavoir.zestwriter.MainApp
Tu préfères que je corrige (pas tout de suite quand même) ou tu veux t'en occuper ? |
je m'en occuperai. 2016-02-18 18:30 GMT+01:00 firm1 notifications@github.com:
|
Bon j'ai corrigé l'erreur et maintenant ça fonctionne. La faiblesse de ma PR est que pour l'instant tout se fait au lancement de l'application. C'est pourquoi je vais devoir faire un autre commit pour que l'initialisation du workspace se fasse au moment où on fait une demande, pas avant. |
Après fetch de ton code, je vois qu'il fonctionne. Je vais tester de mon coté pour voir s'il y'a pas d'effets de bord et le merge suivra. |
le principal problème étant que le filepicker apparaît à l'ouverture de de même rien n'est sauvegardé. Le 21/02/2016 18:55, firm1 a écrit :
|
C'est pas grave, ça ira très bien ainsi pour un début. Au pire, il reste tout la semaine pour l'ajuster. |
en effet Le 21/02/2016 19:00, firm1 a écrit :
|
Ok bon j'ai un peu de mal à faire le truc brut de fonderie. Je fais donc un rebase de la PR et on pourra merger pour que je puisse partir sur une base propre. |
Avant de merger effectivement la sélection au start, c'est un peu trop brut. J'ai intégré le changement de répertoire une fois qu'on est dans l'app. On comprend mieux ce que l'on fait. La prochaine étape sur ce genre de feature sera de sauvegarder l'état |
Ok. Merci pour le merge en tout cas. Le 25/02/2016 08:16, firm1 a écrit :
|
J'ai développé la fonctionnalité de sélection du dossier par filepicking.
Je n'ai pas eu le temps de tester par contre mais comme ça fait un baille que je n'ai pas fait de java je préfère faire la PR maintenant pour voir s'il n'y a pas de gros problèmes.