{"action":"created","issue":{"url":"https://api.github.com/repos/livestreet/livestreet-framework/issues/83","labels_url":"https://api.github.com/repos/livestreet/livestreet-framework/issues/83/labels{/name}","comments_url":"https://api.github.com/repos/livestreet/livestreet-framework/issues/83/comments","events_url":"https://api.github.com/repos/livestreet/livestreet-framework/issues/83/events","html_url":"https://github.com/livestreet/livestreet-framework/issues/83","id":59097806,"number":83,"title":"Переделать логику получения имени таблицы для плагина для ОРМ","user":{"login":"psnet","id":1023918,"avatar_url":"https://avatars.githubusercontent.com/u/1023918?v=3","gravatar_id":"","url":"https://api.github.com/users/psnet","html_url":"https://github.com/psnet","followers_url":"https://api.github.com/users/psnet/followers","following_url":"https://api.github.com/users/psnet/following{/other_user}","gists_url":"https://api.github.com/users/psnet/gists{/gist_id}","starred_url":"https://api.github.com/users/psnet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/psnet/subscriptions","organizations_url":"https://api.github.com/users/psnet/orgs","repos_url":"https://api.github.com/users/psnet/repos","events_url":"https://api.github.com/users/psnet/events{/privacy}","received_events_url":"https://api.github.com/users/psnet/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-26T16:28:16Z","updated_at":"2015-02-26T16:41:56Z","closed_at":null,"body":"Сейчас для использования имени плагина как префикса для его таблиц в орм нужно обязательно указывать соответствие этой таблицы в конфиге (см. https://github.com/livestreet/livestreet-framework/blob/master/classes/engine/MapperORM.class.php#L650-L659 в ```public static function GetTableName($oEntity)```)\r\n```php\r\nif ($sPluginName) {\r\n    $sTablePlugin = $sPluginName . '_' . $sTable;\r\n    /**\r\n     * Для обратной совместимости с 1.0.1\r\n     * Если такая таблица определена в конфиге, то ок, если нет, то используем старый вариант без имени плагина\r\n     */\r\n    if (Config::Get('db.table.' . $sTablePlugin)) {\r\n        $sTable = $sTablePlugin;\r\n    }\r\n}\r\n```\r\nТак вот, *это не дело*. **Нужно стандартизировать все это и чтобы таблицы плагинов всегда имели префикс имени плагина.**\r\n- Это однозначно. Всегда в БД видно какие таблицы к чему относятся: к движку или к какому-то конкретному плагину, а не каша из разных таблиц и *начинай* сравнивать с дампом движка чтобы вспомнить все его таблицы.\r\n- Убирается обязательная запись для этого в конфиге плагина (не нужно для каждой новой таблицы бегать в конфиг и прописывать)\r\n\r\nПоэтому вышеприведенный кусок кода нужно заменить на:\r\n```php\r\n$sTable = $sPluginName ? $sPluginName . '_' . $sTable : $sTable;\r\n```\r\n\r\nтакже можно сравнение ```if (strpos($sEntityName, $sModuleName) === 0) {``` на равенство перевести https://github.com/livestreet/livestreet-framework/blob/master/classes/engine/MapperORM.class.php#L645"},"comment":{"url":"https://api.github.com/repos/livestreet/livestreet-framework/issues/comments/76213047","html_url":"https://github.com/livestreet/livestreet-framework/issues/83#issuecomment-76213047","issue_url":"https://api.github.com/repos/livestreet/livestreet-framework/issues/83","id":76213047,"user":{"login":"mzhelskiy","id":6507185,"avatar_url":"https://avatars.githubusercontent.com/u/6507185?v=3","gravatar_id":"","url":"https://api.github.com/users/mzhelskiy","html_url":"https://github.com/mzhelskiy","followers_url":"https://api.github.com/users/mzhelskiy/followers","following_url":"https://api.github.com/users/mzhelskiy/following{/other_user}","gists_url":"https://api.github.com/users/mzhelskiy/gists{/gist_id}","starred_url":"https://api.github.com/users/mzhelskiy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mzhelskiy/subscriptions","organizations_url":"https://api.github.com/users/mzhelskiy/orgs","repos_url":"https://api.github.com/users/mzhelskiy/repos","events_url":"https://api.github.com/users/mzhelskiy/events{/privacy}","received_events_url":"https://api.github.com/users/mzhelskiy/received_events","type":"User","site_admin":false},"created_at":"2015-02-26T16:41:56Z","updated_at":"2015-02-26T16:41:56Z","body":"Случаи бывают разные, это нужно регламентировать гайдами, а не жестко в коде."}}