summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAntónio Fernandes <antoniof@gnome.org>2020-12-12 20:56:13 +0000
committerOndrej Holy <oholy@redhat.com>2021-01-25 10:54:10 +0000
commitf0759d6fe9bb405c3e9a5a5cf2e9509514f4eb6f (patch)
tree21849d17dd5d45931c46573802ae349d6d437352
parent5c5fee1469d63274c626b1ecb31f5061e40740a9 (diff)
downloadnautilus-f0759d6fe9bb405c3e9a5a5cf2e9509514f4eb6f.tar.gz
file-utilities: Drop nautilus_file_system_is_remote
It compares the filesystem type identifier with a hardcoded list of remote types introduced in 2015.[0] The filesystem::remote attribute has been introduced[1] to handle this, and we have been using it[2], but kept the hardcoded list in order to workaround issues with GLocalFile not setting this attribute correctly. Now that these issues have been fixed[3], we can finally drop it. [0] d3e1a71c6377343af5d61d6ded7bc4f53dfeaf53 [1] https://gitlab.gnome.org/GNOME/glib/-/commit/e57355b055217b067cd1a71340f7ea47e51d4aae [2] 99f55748ef69a68df86b9a039998ee576d59b898 [3] https://gitlab.gnome.org/GNOME/glib/-/merge_requests/1534
-rw-r--r--src/nautilus-file-utilities.c18
-rw-r--r--src/nautilus-file-utilities.h2
-rw-r--r--src/nautilus-file.c11
-rw-r--r--src/nautilus-search-engine.c10
4 files changed, 4 insertions, 37 deletions
diff --git a/src/nautilus-file-utilities.c b/src/nautilus-file-utilities.c
index a3cc97b83..62d8d70bb 100644
--- a/src/nautilus-file-utilities.c
+++ b/src/nautilus-file-utilities.c
@@ -1485,21 +1485,3 @@ location_settings_search_get_recursive_for_location (GFile *location)
return recursive;
}
-
-gboolean
-nautilus_file_system_is_remote (const char *file_system)
-{
- static const gchar * const remote_types[] =
- {
- "afp",
- "google-drive",
- "sftp",
- "webdav",
- "ftp",
- "nfs",
- "cifs",
- NULL
- };
-
- return file_system != NULL && g_strv_contains (remote_types, file_system);
-}
diff --git a/src/nautilus-file-utilities.h b/src/nautilus-file-utilities.h
index 22d851c1b..67df0e4d9 100644
--- a/src/nautilus-file-utilities.h
+++ b/src/nautilus-file-utilities.h
@@ -141,5 +141,3 @@ gchar * nautilus_uri_to_native_uri (const gchar *uri);
NautilusQueryRecursive location_settings_search_get_recursive (void);
NautilusQueryRecursive location_settings_search_get_recursive_for_location (GFile *location);
-
-gboolean nautilus_file_system_is_remote (const char *file_system);
diff --git a/src/nautilus-file.c b/src/nautilus-file.c
index b0dca1e3c..243470432 100644
--- a/src/nautilus-file.c
+++ b/src/nautilus-file.c
@@ -8157,18 +8157,9 @@ nautilus_file_is_in_starred (NautilusFile *file)
gboolean
nautilus_file_is_remote (NautilusFile *file)
{
- g_autofree char *filesystem_type = NULL;
-
g_return_val_if_fail (NAUTILUS_IS_FILE (file), FALSE);
- if (nautilus_file_get_filesystem_remote (file))
- {
- return TRUE;
- }
-
- filesystem_type = nautilus_file_get_filesystem_type (file);
-
- return nautilus_file_system_is_remote (filesystem_type);
+ return nautilus_file_get_filesystem_remote (file);
}
/**
diff --git a/src/nautilus-search-engine.c b/src/nautilus-search-engine.c
index c8505e594..e30f6e007 100644
--- a/src/nautilus-search-engine.c
+++ b/src/nautilus-search-engine.c
@@ -572,16 +572,12 @@ is_recursive_search (NautilusSearchEngineType engine_type,
g_autoptr (GFileInfo) file_system_info = NULL;
file_system_info = g_file_query_filesystem_info (location,
- G_FILE_ATTRIBUTE_FILESYSTEM_TYPE,
+ G_FILE_ATTRIBUTE_FILESYSTEM_REMOTE,
NULL, NULL);
if (file_system_info != NULL)
{
- const char *file_system;
-
- file_system = g_file_info_get_attribute_string (file_system_info,
- G_FILE_ATTRIBUTE_FILESYSTEM_TYPE);
-
- return !nautilus_file_system_is_remote (file_system);
+ return !g_file_info_get_attribute_boolean (file_system_info,
+ G_FILE_ATTRIBUTE_FILESYSTEM_REMOTE);
}
}
}