Skip to content

Commit 8eab205

Browse files
ffissorecmaglie
authored andcommitted
import library also import folder (already expanded libraries)
Conflicts: app/src/processing/app/helpers/FileUtils.java app/test/processing/app/tools/ZipDeflaterTest.java
1 parent 62a8a01 commit 8eab205

File tree

4 files changed

+123
-33
lines changed

4 files changed

+123
-33
lines changed

app/src/processing/app/Base.java

Lines changed: 35 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131

3232
import processing.app.debug.Compiler;
3333
import processing.app.debug.Target;
34+
import processing.app.helpers.FileUtils;
3435
import processing.app.tools.ZipDeflater;
3536
import processing.core.*;
3637
import static processing.app.I18n._;
@@ -2364,24 +2365,43 @@ static protected void listFiles(String basePath,
23642365
}
23652366
}
23662367

2368+
public void handleAddLibrary(Editor editor) {
2369+
JFileChooser fileChooser = new JFileChooser(System.getProperty("user.home"));
2370+
fileChooser.setDialogTitle(_("Select a zip file or a folder containing the library you'd like to add"));
2371+
fileChooser.setFileSelectionMode(JFileChooser.FILES_AND_DIRECTORIES);
23672372

2368-
public void handleAddZipLibrary(Editor editor) {
2369-
String prompt = _("Select a zip file containing the library you'd like to add");
2370-
FileDialog fd = new FileDialog(editor, prompt, FileDialog.LOAD);
2371-
fd.setDirectory(System.getProperty("user.home"));
2372-
fd.setVisible(true);
2373+
Dimension preferredSize = fileChooser.getPreferredSize();
2374+
fileChooser.setPreferredSize(new Dimension(preferredSize.width + 200, preferredSize.height + 200));
2375+
2376+
int returnVal = fileChooser.showOpenDialog(editor);
23732377

2374-
String directory = fd.getDirectory();
2375-
String filename = fd.getFile();
2376-
if (filename == null) return;
2378+
if (returnVal != JFileChooser.APPROVE_OPTION) {
2379+
return;
2380+
}
23772381

2378-
File sourceFile = new File(directory, filename);
2379-
try {
2380-
ZipDeflater zipDeflater = new ZipDeflater(sourceFile, getSketchbookLibrariesFolder());
2381-
zipDeflater.deflate();
2382-
editor.statusNotice(_("Library added to your libraries. Check \"Import library\" menu"));
2383-
} catch (IOException e) {
2384-
editor.statusError(e);
2382+
File sourceFile = fileChooser.getSelectedFile();
2383+
2384+
if (sourceFile.isDirectory()) {
2385+
File destinationFolder = new File(getSketchbookLibrariesFolder(), sourceFile.getName());
2386+
if (!destinationFolder.mkdir()) {
2387+
editor.statusError("Can't create folder: " + sourceFile.getName() + " into libraries folder");
2388+
return;
2389+
}
2390+
try {
2391+
FileUtils.copy(sourceFile, destinationFolder);
2392+
} catch (IOException e) {
2393+
editor.statusError(e);
2394+
return;
2395+
}
2396+
} else {
2397+
try {
2398+
ZipDeflater zipDeflater = new ZipDeflater(sourceFile, getSketchbookLibrariesFolder());
2399+
zipDeflater.deflate();
2400+
} catch (IOException e) {
2401+
editor.statusError(e);
2402+
return;
2403+
}
23852404
}
2405+
editor.statusNotice(_("Library added to your libraries. Check \"Import library\" menu"));
23862406
}
23872407
}

app/src/processing/app/Editor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -631,10 +631,10 @@ public void actionPerformed(ActionEvent e) {
631631
}
632632
sketchMenu.add(importMenu);
633633

634-
item = new JMenuItem(_("Add Library from ZIP"));
634+
item = new JMenuItem(_("Add Library..."));
635635
item.addActionListener(new ActionListener() {
636636
public void actionPerformed(ActionEvent e) {
637-
base.handleAddZipLibrary(Editor.this);
637+
base.handleAddLibrary(Editor.this);
638638
base.onBoardOrPortChange();
639639
base.rebuildImportMenu(Editor.importMenu);
640640
}
Lines changed: 81 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,81 @@
1+
package processing.app.helpers;
2+
3+
import java.io.File;
4+
import java.io.FileInputStream;
5+
import java.io.FileOutputStream;
6+
import java.io.IOException;
7+
8+
public class FileUtils {
9+
10+
/**
11+
* Checks, whether the child directory is a subdirectory of the base directory.
12+
*
13+
* @param base
14+
* the base directory.
15+
* @param child
16+
* the suspected child directory.
17+
* @return true, if the child is a subdirectory of the base directory.
18+
*/
19+
public static boolean isSubDirectory(File base, File child) {
20+
try {
21+
base = base.getCanonicalFile();
22+
child = child.getCanonicalFile();
23+
} catch (IOException e) {
24+
return false;
25+
}
26+
27+
File parentFile = child;
28+
while (parentFile != null) {
29+
if (base.equals(parentFile)) {
30+
return true;
31+
}
32+
parentFile = parentFile.getParentFile();
33+
}
34+
return false;
35+
}
36+
37+
public static void copy(File sourceFolder, File destFolder) throws IOException {
38+
for (File file : sourceFolder.listFiles()) {
39+
File destFile = new File(destFolder, file.getName());
40+
if (file.isDirectory()) {
41+
if (!destFile.mkdir()) {
42+
throw new IOException("Unable to create folder: " + destFile);
43+
}
44+
copy(file, destFile);
45+
} else {
46+
FileInputStream fis = null;
47+
FileOutputStream fos = null;
48+
try {
49+
fis = new FileInputStream(file);
50+
fos = new FileOutputStream(destFile);
51+
byte[] buf = new byte[4096];
52+
int readBytes = -1;
53+
while ((readBytes = fis.read(buf, 0, buf.length)) != -1) {
54+
fos.write(buf, 0, readBytes);
55+
}
56+
} finally {
57+
if (fis != null) {
58+
fis.close();
59+
}
60+
if (fos != null) {
61+
fos.close();
62+
}
63+
}
64+
}
65+
}
66+
}
67+
68+
public static void recursiveDelete(File file) {
69+
if (file.isDirectory()) {
70+
for (File current : file.listFiles()) {
71+
if (current.isDirectory()) {
72+
recursiveDelete(current);
73+
} else {
74+
current.delete();
75+
}
76+
}
77+
}
78+
file.delete();
79+
}
80+
81+
}

app/src/processing/app/tools/ZipDeflater.java

Lines changed: 5 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
import java.util.zip.ZipException;
1111
import java.util.zip.ZipFile;
1212

13+
import processing.app.helpers.FileUtils;
14+
1315
public class ZipDeflater {
1416

1517
private final ZipFile zipFile;
@@ -71,24 +73,11 @@ public void deflate() throws IOException {
7173
private void deleteUndesiredFoldersAndFiles(File folder) {
7274
for (File file : folder.listFiles()) {
7375
if (file.isDirectory() && "__MACOSX".equals(file.getName())) {
74-
recursiveDelete(file);
76+
FileUtils.recursiveDelete(file);
7577
} else if (file.getName().startsWith(".")) {
76-
recursiveDelete(file);
77-
}
78-
}
79-
}
80-
81-
private void recursiveDelete(File file) {
82-
if (file.isDirectory()) {
83-
for (File current : file.listFiles()) {
84-
if (current.isDirectory()) {
85-
recursiveDelete(current);
86-
} else {
87-
current.delete();
88-
}
78+
FileUtils.recursiveDelete(file);
8979
}
9080
}
91-
file.delete();
9281
}
9382

9483
private void ensureFoldersOfEntryExist(File folder, ZipEntry entry) {
@@ -109,7 +98,7 @@ private void ensureOneLevelFolder(File folder) {
10998
}
11099

111100
files[0].renameTo(new File(folder.getParentFile(), files[0].getName()));
112-
recursiveDelete(folder);
101+
FileUtils.recursiveDelete(folder);
113102
}
114103

115104
private String folderNameFromZip() {

0 commit comments

Comments
 (0)