Так как пока что душки-marapper’a, который сие чудо сваял, нету, то отпишу комментариев сюда, а то забуду.
1. строчка 703 – echo $wp_version; – кажется, ты что-то тестовое забыл.
2. “Warning: set_time_limit() [function.set-time-limit]: Cannot set time limit in safe mode in ***/wp-content/plugins/favquotes/favquotes.php on line 986″ – лучше бы аяксами хитрыми импортировать записи, не везде таймлимит выдержит импорт всех фаворитсов.
3. Приятно было бы ещё сделать режим типа [myfavquotes] – для гордого вывода своего лучшего из тви, с показом количества зафаворитивших.
4. Не все ссылки оборачиваются в нофоллов – твиттерные остаются как есть. Так же не помешает галочка “оборачивать всё в noindex” и ставить meta name=”robots” content=”noindex,nofollow”. Да, я подлый латентный СЕОшник.
5. Ещё можно разукорачивать ссылки. Но это уже так, для счастья.
6. Можно выводить гламурные твиттерные аватарки. Не знаю, как они через апи дёргаются, но можно, например, дёргать – и добавить аватар в формат сообщений по дефолту.
7. Хочется упростить заменяемые в форматировании переменные – %author% должно бы отдавать просто юзернейм, без ссылки. Ну или добавить какой-нить %nick% Тогда можно и кнопки на фоллов добавлять, и аватарки тащить как описано в п.6
8. Хороший тон – добавить адрес подцепляемой цсски. Ну, или дать задавать формат пагинатора. Кстати, лучше бы пагинатор сверху и снизу показывался одинаково.
10. bigint в мускульной таблице это ну очень сурово. Инта ансигнед хватит по уши, а бигинт создаёт траблы мускулу ибо имеет виртуальное представление, а на физическое.
11. Аналогично – лучше вместо текстовых полей варчары использовать. Показывает опыт и куча прочитанных мануалов по мускулу -_-
12. Работа с буфером вывода для таких нужд – это ну очень сурово.
13. В функции the_list_quotes прописан global $wp_version; – только для красоты ;)
14. Ну и перевести плагин было бы неплохо – буржуи оценят. Несколько уё#ский тон – показывать твитатник в сайдбаре, твой плагин их бы прокаваил.
15. В качестве параметров не помешал бы и айдишник или адрес цитаты, если они тащатся.
Вот. Это всё, на самом деле, – небольшие придирки, и плагин буду использовать и в текущем виде. Просто приятная штука у тебя получилась, и перфекционист во мне вопит, что её нужно дорабатывать до лоска :)
Ну, ты понимаешь – писалось за пару-тройку часов, специально фор ю :)
2. Да, аякс как бы предполагался, а, скорее всего, даже просто просто очередь. Сейчас проблема с wp_cron какая-то (на локале импортит, на бложеке – нет), так что в любом случае по-другому будет.
8. Формат пагинтора будет.
9. это вообще должен быть нативный скрипт, чтобы в менеджере “выделить все” работало.
6, 7 и 15. Да, %nick% будет. Ссылку на твистатус можно допилить легко – в качестве guid как раз он и берется. Аватары – ну можно, да.
8. CSS’ки встраиваемые терпеть не могу, но что-то будет, да.
4. Ну вот. Не знаю, зачем добавил нофоллоу, теперь придется еще и ноиндекс пилить. Черт.
3. Это придется дергать Диновский фаворитиз, для буржуев такой режим не подойдет – ну нет их почти там, да и простой бложек не справится с самостоятельным нахождением фавов на себя. Так что вряд ли будет.
Буфер появился, когда мне лень стало переделывать эхи.
10. Ты не поверишь, в схеме ВП он почти везде используется, по привычке скопипастил. Хотя, возможно, вообще придется избавляться от авто_инкремента, чтобы упростить импорт.
11. Варчар подойдет под автора, но не под цитату – вообще, он как бы задумывался не только как твицитатник, так что и 255 может быть мало
В общем, думаю, стоит превратить его из цитатника в чистый тви-фаворитс, тогда будет проще. Получится два плагина – банальный и не очень :)
Subscribe? Sure? What for?
Back to top
24 August at 2:53 am
Так как пока что душки-marapper’a, который сие чудо сваял, нету, то отпишу комментариев сюда, а то забуду.
1. строчка 703 – echo $wp_version; – кажется, ты что-то тестовое забыл.
2. “Warning: set_time_limit() [function.set-time-limit]: Cannot set time limit in safe mode in ***/wp-content/plugins/favquotes/favquotes.php on line 986″ – лучше бы аяксами хитрыми импортировать записи, не везде таймлимит выдержит импорт всех фаворитсов.
3. Приятно было бы ещё сделать режим типа [myfavquotes] – для гордого вывода своего лучшего из тви, с показом количества зафаворитивших.
4. Не все ссылки оборачиваются в нофоллов – твиттерные остаются как есть. Так же не помешает галочка “оборачивать всё в noindex” и ставить meta name=”robots” content=”noindex,nofollow”. Да, я подлый латентный СЕОшник.
5. Ещё можно разукорачивать ссылки. Но это уже так, для счастья.
6. Можно выводить гламурные твиттерные аватарки. Не знаю, как они через апи дёргаются, но можно, например, дёргать – и добавить аватар в формат сообщений по дефолту.
7. Хочется упростить заменяемые в форматировании переменные – %author% должно бы отдавать просто юзернейм, без ссылки. Ну или добавить какой-нить %nick% Тогда можно и кнопки на фоллов добавлять, и аватарки тащить как описано в п.6
8. Хороший тон –
добавить адрес подцепляемой цсски. Ну, илидать задавать формат пагинатора. Кстати, лучше бы пагинатор сверху и снизу показывался одинаково.9. Строчка 97 плагина. “script type=’text/javascript’ src=’http://localhost/blog/wp-admin/js/forms.js?ver=20080401″ о_О
10. bigint в мускульной таблице это ну очень сурово. Инта ансигнед хватит по уши, а бигинт создаёт траблы мускулу ибо имеет виртуальное представление, а на физическое.
11. Аналогично – лучше вместо текстовых полей варчары использовать. Показывает опыт и куча прочитанных мануалов по мускулу -_-
12. Работа с буфером вывода для таких нужд – это ну очень сурово.
13. В функции the_list_quotes прописан global $wp_version; – только для красоты ;)
14. Ну и перевести плагин было бы неплохо – буржуи оценят. Несколько уё#ский тон – показывать твитатник в сайдбаре, твой плагин их бы прокаваил.
15. В качестве параметров не помешал бы и айдишник или адрес цитаты, если они тащатся.
Вот. Это всё, на самом деле, – небольшие придирки, и плагин буду использовать и в текущем виде. Просто приятная штука у тебя получилась, и перфекционист во мне вопит, что её нужно дорабатывать до лоска :)
24 August at 9:59 am
Ну, ты понимаешь – писалось за пару-тройку часов, специально фор ю :)
2. Да, аякс как бы предполагался, а, скорее всего, даже просто просто очередь. Сейчас проблема с wp_cron какая-то (на локале импортит, на бложеке – нет), так что в любом случае по-другому будет.
8. Формат пагинтора будет.
9. это вообще должен быть нативный скрипт, чтобы в менеджере “выделить все” работало.
6, 7 и 15. Да, %nick% будет. Ссылку на твистатус можно допилить легко – в качестве guid как раз он и берется. Аватары – ну можно, да.
8. CSS’ки встраиваемые терпеть не могу, но что-то будет, да.
4. Ну вот. Не знаю, зачем добавил нофоллоу, теперь придется еще и ноиндекс пилить. Черт.
3. Это придется дергать Диновский фаворитиз, для буржуев такой режим не подойдет – ну нет их почти там, да и простой бложек не справится с самостоятельным нахождением фавов на себя. Так что вряд ли будет.
Буфер появился, когда мне лень стало переделывать эхи.
10. Ты не поверишь, в схеме ВП он почти везде используется, по привычке скопипастил. Хотя, возможно, вообще придется избавляться от авто_инкремента, чтобы упростить импорт.
11. Варчар подойдет под автора, но не под цитату – вообще, он как бы задумывался не только как твицитатник, так что и 255 может быть мало
В общем, думаю, стоит превратить его из цитатника в чистый тви-фаворитс, тогда будет проще. Получится два плагина – банальный и не очень :)
24 August at 10:34 am
3. А из фаворитез нельзя вытащить свои сообщения?
10. Да вордпресс вообще не идеал по базе и коду %)
А несовместимость базы вообще не волнует – надо будет просто заново засосать одним кликом :)
24 August at 10:04 am
Правда, есть проблема – новая версия будет несовместима по базе со старой %)