Provide an API function to convert a URI to a file object

Looks good but I think we really need an additional API function that will take a URI and return the existing file object, or save as a new one otherwise.

yes, otherwise, sending it an existing youtube url (for instance) will fail. as will uploading a file that already exists (is there an md5 check somewhere in the system?)

ah, scratch that re the md5; it's the filename doing that, i think...

pretty sure that this will choke on identical filenames as well.

No more new api's in D7, so bumping to D8.

this addresses some of the concerns in #3 and 4:

this will load a file object if the uri exists in the db. otherwise, it creates a new object, and sets a ->is_new flag on the new object. doesn't actually save it, as that rightfully already exists in file_save().

for reference, the media module is also using this function (and struggling with the issues here); see #1023254: Load a media object from a given URI.

Subscribe.

Major missing functionality. For those needing this in D7, the media module includes this function.

Subscribe.

the file_styles module has now also implemented it's own version of this : file_styles_uri_to_object

+++ includes/file.inc	13 Jan 2011 16:24:47 -0000
@@ -2057,6 +2057,41 @@ function file_get_mimetype($uri, $mappin
+  $query = db_select('file_managed', 'f')
+    ->fields('f', array('fid'))
+    ->condition('uri', $uri)
+    ->execute()
+    ->fetchCol();
+  if (!empty($query)) {
+    $file = file_load(array_shift($query));
+  }

Could be condensed just using $files = entity_load('file', FALSE, array('uri' => $uri)); $file = !empty($files) ? reset($files) : FALSE;

Should also run $uri = file_stream_wrapper_uri_normalize($uri) prior to searching for an existing file.

+++ includes/file.inc	13 Jan 2011 16:24:47 -0000
@@ -2057,6 +2057,41 @@ function file_get_mimetype($uri, $mappin
+    $wrapper = file_stream_wrapper_get_instance_by_uri($uri);

$wrapper variable is not used. Remove this line.

+++ includes/file.inc	13 Jan 2011 16:24:47 -0000
@@ -2057,6 +2057,41 @@ function file_get_mimetype($uri, $mappin
+    $file->is_new = TRUE;

This is unnecessary as checking if empty($file->fid) is a new file.

19 days to next Drupal core point release.

Subscribe.

Rerolled #9 with fixes suggested by Dave Reid in #15.

The patch uses $file = new StdClass;

I don't know if we have a documented standard for this, but everywhere else in core we use $file = new stdClass();

Good eye. Rerolled for consistency.

function file_uri_to_object($uri)
code below:
codigo abaixo:
===
pt-BR
PS: na linha 51 add } eu adicioneu essa tag aqui = }
rodei a atualização o erro saiu, isso esta correto?
--//--
pt-EN
PS: at line 51 add} tag I added this here =}
I ran the update the error came out, this is correct?
===

will know much about programming, so I'm taking the entirely the code that is running on my work project completion of course (CBT).
analysts so that you can take a look I just made a change and it worked, at least out of the error tela.espero not be wrong. ORIGINAL en, sorry my English.

ORIGINAL aqui-

não entendo muito de programação, então estou pegando o codigo enteiro que está rodando no meu projeto de trabalho de conclusão de curso (TCC) .
para que vocês analistas possam dar uma olhada eu fiz apenas uma modificação e funcionou, pelo menos o erro saiu da tela.espero não estar errado. ORIGINAL pt-BR , sorry my english :-( .

( - is equals / , ok )
code below:Atatach

Marked #367121: Wrapper function to turn static file into a file object as a duplicate of this issue even though it was older, this had more work on it.

#19: 685818-file-uri-to-object-rev2.patch queued for re-testing.

There has been no movement on this for a while. Before doing looking at closer thought a re test was in order.

This is just a reroll of the patch from #19

I have tweaked the function a little and changed the core file functions to use it. With the exception of file_scan_directory() which really didn't like using it.

Although I think if #1361226: Make the file entity a classed object happens this will be moot.

Do we still need this? Now core provides entity_create(), which is used by file_save_upload() and file_save_data().

According to entity_load the third parameter you are using in the path here is deprecated.

Yes this is still needed. Because file_save_data() in core still duplicates the code that all the contrib modules will end up duplicating in D8 as well.

After some investigation, I believe this issue can be closed because D8 already has satisfied this request.

In file_save_data():

  if ($uri = file_unmanaged_save_data($data, $destination, $replace)) {
    // Create a file entity.
    $file = File::create([
      'uri' => $uri,
      'uid' => $user->id(),
      'status' => FILE_STATUS_PERMANENT,
    ]);

In file_save_upload():

    // Begin building file entity.
    $values = array(
      'uid' => $user->id(),
      'status' => 0,
      'filename' => $file_info->getClientOriginalName(),
      'uri' => $file_info->getRealPath(),
      'filesize' => $file_info->getSize(),
    );
    $values['filemime'] = \Drupal::service('file.mime_type.guesser')->guess($values['filename']);
    $file = File::create($values);

In Drupal\file\Entity\File:

   /**
    * {@inheritdoc}
    */
   public static function preCreate(EntityStorageInterface $storage, array &$values) {
     // Automatically detect filename if not set.
     if (!isset($values['filename']) && isset($values['uri'])) {
       $values['filename'] = drupal_basename($values['uri']);
     }

     // Automatically detect filemime if not set.
     if (!isset($values['filemime']) && isset($values['uri'])) {
       $values['filemime'] = \Drupal::service('file.mime_type.guesser')->guess($values['uri']);
     }
   }

So at least we shouldn't continue on D8.

On the other hand, as with D7, I think we can add new APIs currently. Please reopen this if you want this kind of function in D7.

The major part of this is ability to re-use an existing file (either a permanent file or a temporary file owned by the current user), which is not addressed in #34. I do not consider this resolved.

Also, if an issue is still valid for D7, let's please not just close it, change the version and update it. That said, this is still valid for D8. It would be useful to have it as a method on the File entity object itself.

This is what I've needed to re-use on several projects.

@Dave Reid, thank you for your response. I got it.

+++ b/core/modules/file/src/Entity/File.php
@@ -277,4 +277,38 @@ public static function baseFieldDefinitions(EntityTypeInterface $entity_type) {
+        return $file->isPermanent() || $file->getOwnerId() == \Drupal::currentUser()

getOwnerId() may return null. It needs type check.

Patch in #37 is no longer applicable. Needs reroll.

Plain reroll.

Closing this as a duplicate of #3223209: deprecate file_save_data, file_copy and file_move and replace with a service
Transferring issue credit over there

Changing to a won't fix as the other issue removed it again and with the loadByUri() that we *do* add, it just doesn't seem worth adding that on its own anymore. There has been little to no interest here in years.