preserve order during opml import (#707)

src/main/java/com/commafeed/backend/opml/OPMLImporter.java

@@ -38,8 +38,8 @@ public class OPMLImporter {
 		try {
 			Opml feed = (Opml) input.build(new StringReader(xml));
 			List<Outline> outlines = feed.getOutlines();
-			for (Outline outline : outlines) {
+			for (int i = 0; i < outlines.size(); i++) {
-				handleOutline(user, outline, null);
+				handleOutline(user, outlines.get(i), null, i);
 			}
 		} catch (Exception e) {
 			log.error(e.getMessage(), e);
@@ -47,7 +47,7 @@ public class OPMLImporter {
 
 	}
 
-	private void handleOutline(User user, Outline outline, FeedCategory parent) {
+	private void handleOutline(User user, Outline outline, FeedCategory parent, int position) {
 		List<Outline> children = outline.getChildren();
 		if (CollectionUtils.isNotEmpty(children)) {
 			String name = FeedUtils.truncate(outline.getText(), 128);
@@ -64,11 +64,12 @@ public class OPMLImporter {
 				category.setName(name);
 				category.setParent(parent);
 				category.setUser(user);
+				category.setPosition(position);
 				feedCategoryDAO.saveOrUpdate(category);
 			}
 
-			for (Outline child : children) {
+			for (int i = 0; i < children.size(); i++) {
-				handleOutline(user, child, category);
+				handleOutline(user, children.get(i), category, i);
 			}
 		} else {
 			String name = FeedUtils.truncate(outline.getText(), 128);
@@ -80,7 +81,7 @@ public class OPMLImporter {
 			}
 			// make sure we continue with the import process even if a feed failed
 			try {
-				feedSubscriptionService.subscribe(user, outline.getXmlUrl(), name, parent);
+				feedSubscriptionService.subscribe(user, outline.getXmlUrl(), name, parent, position);
 			} catch (FeedSubscriptionException e) {
 				throw e;
 			} catch (Exception e) {

src/main/java/com/commafeed/backend/service/FeedSubscriptionService.java

@@ -44,7 +44,15 @@ public class FeedSubscriptionService {
 	private final CacheService cache;
 	private final CommaFeedConfiguration config;
 
-	public Feed subscribe(User user, String url, String title, FeedCategory category) {
+	public Feed subscribe(User user, String url, String title) {
+		return subscribe(user, url, title, null, 0);
+	}
+
+	public Feed subscribe(User user, String url, String title, FeedCategory parent) {
+		return subscribe(user, url, title, parent, 0);
+	}
+
+	public Feed subscribe(User user, String url, String title, FeedCategory category, int position) {
 
 		final String pubUrl = config.getApplicationSettings().getPublicUrl();
 		if (StringUtils.isBlank(pubUrl)) {
@@ -63,7 +71,7 @@ public class FeedSubscriptionService {
 			sub.setUser(user);
 		}
 		sub.setCategory(category);
-		sub.setPosition(0);
+		sub.setPosition(position);
 		sub.setTitle(FeedUtils.truncate(title, 128));
 		feedSubscriptionDAO.saveOrUpdate(sub);
 

src/main/java/com/commafeed/frontend/resource/FeedREST.java

@@ -395,7 +395,7 @@ public class FeedREST {
 			url = fetchFeedInternal(url).getUrl();
 
 			FeedInfo info = fetchFeedInternal(url);
-			feedSubscriptionService.subscribe(user, info.getUrl(), info.getTitle(), null);
+			feedSubscriptionService.subscribe(user, info.getUrl(), info.getTitle());
 		} catch (Exception e) {
 			log.info("Could not subscribe to url {} : {}", url, e.getMessage());
 		}

src/test/java/com/commafeed/backend/opml/OPMLImporterTest.java

@@ -46,7 +46,7 @@ public class OPMLImporterTest {
 		importer.importOpml(user, xml);
 
 		Mockito.verify(feedSubscriptionService).subscribe(Mockito.eq(user), Mockito.anyString(), Mockito.anyString(),
-				Mockito.any(FeedCategory.class));
+				Mockito.any(FeedCategory.class), Mockito.anyInt());
 	}
 
 }