Добрый день.
Предлагаю разделить модуль на два модуля: один отвечает за добавления счетчика на сайт (с настройками по страницам/ролям), другой, зависящий от первого, добавляет в админку статистику из Яндекс.Материки.
Не всем нужна эта статистика в админке, к тому же зависящая от другого модуля (Chart).
По аналогии с модулем Google Analytics и Google Analytics Reports. О целесообразности этого можно посмотреть по количеству сайтов, использующих эти модули.
---
Description of the maintainer:
Several users requested splitting of functionality into two parts, such as:
1) Module for counter code manipulations;
2) Module for application authorization and reports displaying.
The motivation:
1) The reports depend on other module - chart. The users don't want to install any other modules.
2) The reports are not necessary for some users.
I've decided to split the code of project into two separate modules within existing project.
Comments
Comment #1
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedПривет,
Ты уже не первый ))
http://komelin.com/content/modul-yandexmetrics-obnovlen-do-6x-13-i-7x-12...
Я уже работаю надо ослаблением связи между двумя частями, вот, например, убрал редирект и изменил README: http://drupal.org/node/1487828
И, да, мысль хорошая. Можно будет заняться этим после более важных тикетов.
Comment #2
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedНачал работу по разделению.
--
Started splitting.
Comment #3
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedComment #4
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedHi there,
I'm happy to report I worked hard and separated counter from reports.
Now the project includes two modules, such as: Yandex.Metrics Counter (the main) and Yandex.Metrics Reports (related).
Here you are dev releases:
yandex_metrics 6.x-2.x-dev
yandex_metrics 7.x-2.x-dev
Please test and report here about upgrading experience.
How to upgrade from 6.x-1.x and 7.x-1.x:
I recommend to remove all content from the module directory and copy new files there.
Then run /update.php to perform database changes.
---
Привет,
Рад сообщить, что я усердно поработал и отделил счетчик от отчетов.
Теперь проект включает 2 модуля: Yandex.Metrics Counter (главный) и Yandex.Metrics Reports (связаный с ним под-модуль).
Здесь девелоперские релизы:
yandex_metrics 6.x-2.x-dev
yandex_metrics 7.x-2.x-dev
Пожалуйста, тестируйте и пишите в этом тикете как прошло обновление.
Как обновлять с версий 6.x-1.x и 7.x-1.x:
Я рекомендую удалить все содержимое папки с проектом и скопировать туда новые файлы.
Потом запустить /update.php, чтобы применить изменения в базе данных.
Comment #5
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedComment #6
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedLooks good.
Comment #7
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedIt is necessary to change README.txt
Comment #8
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedCorrected documentation for D6 according to the ticket changes.
Commit: http://drupalcode.org/project/yandex_metrics.git/commit/2db70d5
Comment #9
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedPending task here is documentation for 7.x-2.x
Comment #10
kalabroLet's try tag system.
Comment #11
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedCool. Great suggestion!
Comment #12
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedWhat do you think if we split our README.txt into README.txt (English) and README.RU.txt (Russian)?
It's clearer for users.
Comment #13
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedKate, please review README.txt.
I've split the file by languages. I'll update Russian file later when English will be completed.
Comment #14
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedComment #14.0
Konstantin Komelin CreditAttribution: Konstantin Komelin commentedAdded English description.