Issue #2010216 by kgoel, hussainweb: Convert picture_mapping_uri() to ->uri().

core/modules/picture/lib/Drupal/picture/Plugin/Core/Entity/PictureMapping.php

@@ -30,7 +30,6 @@ use Drupal\picture\PictureMappingInterface;
  *     }
  *   },
  *   list_path = "admin/config/media/picturemapping",
- *   uri_callback = "picture_mapping_uri",
  *   config_prefix = "picture.mappings",
  *   entity_keys = {
  *     "id" = "id",
@@ -163,4 +162,16 @@ class PictureMapping extends ConfigEntityBase implements PictureMappingInterface
     return $mapping_found;
   }
 
+  /**
+   * {@inheritdoc}
+   */
+  public function uri() {
+    return array(
+      'path' => 'admin/config/media/picturemapping/' . $this->id(),
+      'options' => array(
+        'entity_type' => $this->entityType,
+        'entity' => $this,
+      ),
+    );
+  }
 }

core/modules/picture/picture.module

@@ -116,24 +116,6 @@ function picture_mapping_load($id) {
   return entity_load('picture_mapping', $id);
 }
 
-/**
- * Gets Picture uri callback.
- */
-function picture_mapping_uri(PictureMapping $picture_mapping) {
-  return array(
-    'path' => 'admin/config/media/picturemapping/' . $picture_mapping->id(),
-  );
-}
-
-/**
- * Sets Picture uri callback.
- */
-function picture_mapping_set_uri(PictureMapping $picture_mapping) {
-  return array(
-    'path' => 'admin/config/media/picturemapping/' . $picture_mapping->id(),
-  );
-}
-
 /**
  * Implements hook_theme().
  */