I have constructed an API service that dives into a storage engine and retrieve data.I have tried my best to apply the MVC modelling with services (repository) in mind where the controller (as it should) is just the data direction (input goes in and it redirects the output with the Repo in the middle doing all the heavy lifting).The framework is Silex (the small version of Symfony).For all purpose in keeping the code simple - I'm only going to paste 1 core controller function rather than everything (as it is not needed to comprehend). If you need more info - do not hesitate in asking.Currently I have 2 version of the same controller so that I can test the end results(bad practice I am aware, as I am suppose to use an interface to inject the specific factory object in order to use the correct repository... except I'm having a small problem wrapping my head around it so I choose to have 2 controllers).The first version - is DB driven (using Eloquent from Laravel) - the second version is using ElasticSearch. There will be a third version using a NoSQL (which is why I want to efficiently refactor the MVC to be extensible).Please keep in mind for any recommendation/answer given that I will be plugging in a future storage engine.Version 1 DB Drivenclass MusicController extends BaseController{    const MUSIC_REPOSITORY_CLASS_NAME = '\Api\Manager\Media\Music\MusicRepository';    const MUSIC_MEDIA_NAME            = 'music_album';    const MUSIC_MEDIA_ENTITY_NAME     = '\Api\Entity\Music\Album';/** * __constructor to save Application Container. * * @param Application $app Application Container. * */public function __construct(Application $app){    parent::__construct($app);}public function getPopularListAction($device, $limit, $offset){    $musicRepo = new MusicRepository();    $where     = array(        'mediaTypeId'    => $musicRepo::MEDIA_TYPE_MUSIC_ID,        'mediaTypeName'  => $musicRepo::MEDIA_TYPE_MUSIC_NAME,        'device'         => $device,        'status'         => 'active',    );    $albums  = $musicRepo->getByPopularity(        self::MUSIC_MEDIA_ENTITY_NAME,        $where,        $limit,        $offset    );    return $albums;}}class MusicRepository{public function getByPopularity($mediaEntityName, $where, $limit, $offset){    $mediaObject = new $mediaEntityName();    $mediaTableName = $mediaObject->getTable();    $mediaTypeName = $where['mediaTypeName'];    $album = $mediaObject        ->selectRaw('music_album.*, count(music.id) as songs_count, score')        ->SongsWithInnerJoin()        ->with('artists')        ->with('provider')        ->having('songs_count', '>', '0')        ->take($limit)        ->skip($offset)        ->orderBy('score', 'DESC');    $returnResult = $album->get()->toArray();    return $returnResult;}}class Album extends Eloquent{/** * @var string  $table      music_album * @var string  $primaryKey id * @var bool    $timestamps no timestamp autosave */protected $table      = music_album;protected $primaryKey = 'id';public $timestamps    = false;/** * Return hasMany relationship with Music Entity ordered by id * @return mixed */public function songs(){    return $this->hasMany('Api\Entity\Music\Music','album_id','id')->OrderBy('id','ASC');}/** * Return hasMany relationship with Music Entity ordered by id * @return mixed */public function artists(){    return $this->belongsToMany(        'Api\Entity\Music\Artist',        'music_album_artists',        'album_id',        'artist_id'    );}/** * Inner Join with music table * @param $query    current query */public function scopeSongsWithInnerJoin($query){    $query->join('music as m', 'm.album_id', '=', 'music_album.id');}}VERSION 2 - ElasticSearch Storage Engineclass BaseController{protected $app;public function __construct(Application $app){    $this->app = $app;}public static function after($content){    $esOutput = new OutputFormat($content);    $normalizedContent = $esOutput->formatData();    $content = array(        'statusCode' => 200,        'content'    => $normalizedContent    );    return $content;}}class MusicController extends BaseController{const MUSIC_REPOSITORY_CLASS_NAME = '\Api\Manager\Media\Music\MusicRepository';const MEDIA_ELASTIC_REPOSITORY_CLASS_NAME = '\Api\Manager\Media\v2\ElasticMediaRepository';const MEDIA_REPOSITORY_CLASS_NAME = '\Api\Manager\Media\v2\MediaRepository';const MUSIC_MEDIA_NAME = 'music_album';const MUSIC_MEDIA_ENTITY_NAME = '\Api\Entity\Music\Album';/** * Fetch Music Album sorted by Popularity * * @param $device   device type id * @param $limit    max number of results returned * @param $offset   starting index of result * * @return mixed */public function getPopularListAction($device, $limit, $offset){    $mediaSearchRepo = new ElasticRepository();    $where = array(        'mediaTypeId'   => MUSIC_REPOSITORY_CLASS_NAME::MEDIA_TYPE_MUSIC_ID,        'mediaTypeName' => MUSIC_REPOSITORY_CLASS_NAME::MEDIA_TYPE_ALBUM_NAME,        'device'        => $device,        'status'        => 'active',    );    $result = $mediaSearchRepo->getPopularList($where, $limit, $offset);    return $result;}}class ElasticRepository{public function getPopularList($where, $limit, $offset){    $config = \Lib\Config::get('elasticsearch');    $es = new \Elasticsearch\Client(array('hosts'=>array($config['server'])));    $mediaType = $where['mediaTypeName'];    $extraFilter = '';    if ($mediaType !== null) {        $extraFilter = '              ,{                term: {                        media_type: {                            value: '.$mediaType.'                        }                    }              }            ';    }    $json = '{                    query: {                     filter: {                        bool: {                           must: [                              {                                 term: {                                    status: {                                       value: active                                    }                                 }                              }                              '.$extraFilter.'                           ]                        }                     }                  }               },               sort: [                  {                      _score:{                        order: desc                      },                     score: {                        order: desc                     }                  }               ],                from:'.$offset.',                size:'.$limit.'            }        ';    $params['index'] = $config['index'];    $params['type'] = $config['type'];    $params['body'] = $json;    $result = $es->search($params);    if (isset($result['hits']['hits']['0']) && isset($config['minRelevanceScoreToReturn'])) {        if ($result['hits']['hits']['0']['_score'] < $config['minRelevanceScoreToReturn']) {            return false;        }    }    return $result['hits']['hits'];}}class OutputFormat{protected $originalData = null;protected $outputData = null;public function __construct($data = array()){    $this->originalData = $data;}public function formatData(){    if ($this->originalData == null) {        return $this->outputData;    }    $this->outputData = $this->originalData;    if(is_array($this->originalData)) {        if (isset($this->originalData[0]->_source)) {            $this->outputData = array();            foreach($this->originalData as $data) {                $this->outputData[] = $data->_source;            }        }    }    return $this->outputData;}}Since this is an API - Ideally I would like it to be returned in an array format so that in the view it is shown as a json.Notes:Version 1 - its Eloquent (from laravel) driven - the object is stemming from the DB's fields - So to me when creating Album makes sense to extend it from Eloquent to retrieve and let the Repository do the querying (and not the controller).However clearly as shown in version 2 - the Repository driven queries are now from ElasticSearch and the model is no longer usable.If I run two different end points - these will work. 1 thing that irks me but I couldn't find a way to optimize it efficiently: OutputFormat class that needs to iterate in order to normalize the output (ElasticSearch toss into their output _source on each object returned which is a nuissance to the end user).Questions:Given the working structure, was the best approach of MVC in order to achieve uniformity and also keep SOLID + DRY? I clearly fail on the dry since I had to make 2 controllers.As mentioned above - I'm going to insert a third storage source, so combining and refactoring this would need to be extensible for two future tweaks:future storage engine to be able to plug in and play.If I have a controller - that may require to utilize all 3 different storage engine for different purpose - the repository (or service) method makes the most sense since the controller feeds in input and receives output.However, given that a Data Model shouldn't know about databases because it renders it inefficient, I'm uncertain about optimizing the above code for it to conform to that concept.Note: I've taken an extensive read to @Elias van Ootegem's post, however I'm not 100% sure how to clean and optimize the above code
Data model in MVC for multiple providers
php;object oriented;mvc