From 75fc133d7756df5ac29d66e677bde312a921f859 Mon Sep 17 00:00:00 2001
From: Peter Andersson <pelleplutt1976@gmail.com>
Date: Tue, 10 Feb 2015 22:24:23 +0100
Subject: [PATCH] added rename func

---
 src/spiffs.h             |  8 ++++++++
 src/spiffs_hydrogen.c    | 40 ++++++++++++++++++++++++++++++++++++++++
 src/test/test_hydrogen.c | 27 +++++++++++++++++++++++++++
 3 files changed, 75 insertions(+)

diff --git a/src/spiffs.h b/src/spiffs.h
index 2d82a41..17f3d77 100644
--- a/src/spiffs.h
+++ b/src/spiffs.h
@@ -371,6 +371,14 @@ s32_t SPIFFS_fflush(spiffs *fs, spiffs_file fh);
  */
 void SPIFFS_close(spiffs *fs, spiffs_file fh);
 
+/**
+ * Renames a file
+ * @param fs            the file system struct
+ * @param old           path of file to rename
+ * @param new           new path of file
+ */
+s32_t SPIFFS_rename(spiffs *fs, char *old, char *new);
+
 /**
  * Returns last error of last file operation.
  * @param fs            the file system struct
diff --git a/src/spiffs_hydrogen.c b/src/spiffs_hydrogen.c
index af7d8c0..b8b3373 100644
--- a/src/spiffs_hydrogen.c
+++ b/src/spiffs_hydrogen.c
@@ -617,6 +617,46 @@ void SPIFFS_close(spiffs *fs, spiffs_file fh) {
   SPIFFS_UNLOCK(fs);
 }
 
+s32_t SPIFFS_rename(spiffs *fs, char *old, char *new) {
+  SPIFFS_API_CHECK_MOUNT(fs);
+  SPIFFS_LOCK(fs);
+
+  spiffs_page_ix pix_old, pix_dummy;
+  spiffs_fd *fd;
+
+  s32_t res = spiffs_object_find_object_index_header_by_name(fs, (u8_t*)old, &pix_old);
+  SPIFFS_API_CHECK_RES_UNLOCK(fs, res);
+
+  res = spiffs_object_find_object_index_header_by_name(fs, (u8_t*)new, &pix_dummy);
+  if (res == SPIFFS_ERR_NOT_FOUND) {
+    res = SPIFFS_OK;
+  } else if (res == SPIFFS_OK) {
+    res = SPIFFS_ERR_CONFLICTING_NAME;
+  }
+  SPIFFS_API_CHECK_RES_UNLOCK(fs, res);
+
+  res = spiffs_fd_find_new(fs, &fd);
+  SPIFFS_API_CHECK_RES_UNLOCK(fs, res);
+
+  res = spiffs_object_open_by_page(fs, pix_old, fd, 0, 0);
+  if (res != SPIFFS_OK) {
+    spiffs_fd_return(fs, fd->file_nbr);
+  }
+  SPIFFS_API_CHECK_RES_UNLOCK(fs, res);
+
+  res = spiffs_object_update_index_hdr(fs, fd, fd->obj_id, fd->objix_hdr_pix, 0, (u8_t*)new,
+      0, &pix_dummy);
+
+  if (res != SPIFFS_OK) {
+    spiffs_fd_return(fs, fd->file_nbr);
+  }
+  SPIFFS_API_CHECK_RES_UNLOCK(fs, res);
+
+  SPIFFS_UNLOCK(fs);
+
+  return res;
+}
+
 spiffs_DIR *SPIFFS_opendir(spiffs *fs, char *name, spiffs_DIR *d) {
   (void)name;
   if (!SPIFFS_CHECK_MOUNT(fs)) {
diff --git a/src/test/test_hydrogen.c b/src/test/test_hydrogen.c
index eca56b2..b0cbfa1 100644
--- a/src/test/test_hydrogen.c
+++ b/src/test/test_hydrogen.c
@@ -291,6 +291,33 @@ TEST(open_by_dirent) {
 
 } TEST_END(open_by_dirent)
 
+
+TEST(rename) {
+  int res;
+
+  char *src_name = "baah";
+  char *dst_name = "booh";
+  char *dst_name2 = "beeh";
+  int size = SPIFFS_DATA_PAGE_SIZE(FS);
+
+  res = test_create_and_write_file(src_name, size, size);
+  TEST_CHECK(res >= 0);
+
+  res = SPIFFS_rename(FS, src_name, dst_name);
+  TEST_CHECK(res >= 0);
+
+  res = SPIFFS_rename(FS, dst_name, dst_name);
+  TEST_CHECK(res < 0);
+  TEST_CHECK(SPIFFS_errno(FS) == SPIFFS_ERR_CONFLICTING_NAME);
+
+  res = SPIFFS_rename(FS, src_name, dst_name2);
+  TEST_CHECK(res < 0);
+  TEST_CHECK(SPIFFS_errno(FS) == SPIFFS_ERR_NOT_FOUND);
+
+  return TEST_RES_OK;
+} TEST_END(rename)
+
+
 TEST(remove_single_by_path)
 {
   int res;