Remove some todos

src/Contracts/Feature.php

@@ -10,6 +10,5 @@ use Stancl\Tenancy\TenantManager;
 // todo should this be FeatureProvider?
 interface Feature
 {
-    // todo is the tenantManager argument necessary?
     public function bootstrap(TenantManager $tenantManager): void;
 }

src/Contracts/StorageDriver.php

@@ -9,9 +9,9 @@ use Stancl\Tenancy\Exceptions\TenantCouldNotBeIdentifiedException;
 
 interface StorageDriver
 {
-    public function createTenant(Tenant $tenant): bool; // todo return type
+    public function createTenant(Tenant $tenant): bool;
 
-    public function updateTenant(Tenant $tenant): bool; // todo return type
+    public function updateTenant(Tenant $tenant): bool;
 
     /**
      * Find a tenant using an id.

src/Contracts/TenancyBootstrapper.php

@@ -8,7 +8,6 @@ use Stancl\Tenancy\Tenant;
 
 interface TenancyBootstrapper
 {
-    public function start(Tenant $tenant); // todo better name?
+    public function start(Tenant $tenant);
-
+    public function end();
-    public function end(Tenant $tenant); // todo arg?
 }

src/TenancyBootstrappers/QueueTenancyBootstrapper.php

@@ -21,7 +21,7 @@ class QueueTenancyBootstrapper implements TenancyBootstrapper
         $this->app['queue']->createPayloadUsing([$this, 'createPayload']);
         $this->app['events']->listen(\Illuminate\Queue\Events\JobProcessing::class, function ($event) {
             if (\array_key_exists('tenant_id', $event->job->payload())) {
-                tenancy()->initById($event->job->payload()['tenant_id']); // todo
+                tenancy()->initById($event->job->payload()['tenant_id']);
             }
         });
     }

src/Tenant.php

@@ -7,7 +7,6 @@ namespace Stancl\Tenancy;
 use ArrayAccess;
 
 // todo tenant storage
-// todo create tenant - create db?
 
 /**
  * @internal Class is subject to breaking changes in minor and patch versions.

src/TenantModel.php

@@ -6,25 +6,16 @@ namespace Stancl\Tenancy;
 
 use Illuminate\Database\Eloquent\Model;
 
-// todo move this to a database driver domain?
-
 /**
  * @internal Class is subject to breaking changes in minor and patch versions.
  */
 class TenantModel extends Model
 {
     protected $guarded = [];
-    protected $primaryKey = 'uuid';
+    protected $primaryKey = 'id';
     public $incrementing = false;
     public $timestamps = false;
 
-    /**
-     * Decoded data from the data column.
-     *
-     * @var object
-     */
-    private $dataObject;
-
     public static function dataColumn()
     {
         return config('tenancy.storage.db.data_column', 'data');
@@ -87,10 +78,13 @@ class TenantModel extends Model
         return $this->attributes[$key] ?? $this->getFromData($key) ?? null;
     }
 
-    /** @todo In v2, this should return an associative array. */
     public function getMany(array $keys): array
     {
-        return \array_map([$this, 'get'], $keys);
+        return array_reduce($keys, function ($result, $key) {
+            $result[$key] = $this->get[$key];
+
+            return $result;
+        }, []);
     }
 
     public function put(string $key, $value)