Вы здесьКак нам обустроить заливку KIT-4063
Опубликовано чт, 28/08/2008 - 16:51 пользователем pkn
Как нам обустроить заливку KIT-4063 При имеющемся коде заливочного скрипта (file author.inc, function AddFileToLibrusec) всем этим 4063 книгам при заливке будут обнаружены дубли, конфликт которых придется разрешать вручную. Это очень много кликов. За один день вручную столько не сделать, и получится, что эта заливка забьет ленту новостей на несколько дней в лучшем случае. Да и лень делать вручную то, что можно отдать машине, она железная. Поэтому я хочу попросить Ларина внести изменения в заливочный скрипт. Точный текст изменения я сформирую попозже, а пока смысл этого изменения. Как работает заливочный скрипт сейчас: Как я предлагаю: 1. Автоматический апгрейд той же книги Если 2. Автоматическая замена книги, сделанной LibRusEc kit Если Если ни одна из этих проверок не выполняется - то, как и раньше, предложение разрешить конфликт дублей вручную. Вопрос: не создаст ли такое изменение каких-либо подводных камней? ----------- -----------
|
Вход на сайтПоиск по блогам и форумамUser menuПоследние комментарии
babajga RE:Сказки без подсказки. Сборник 15 часов
ostap79 RE:Прошу переформатировать, распознать, etc... 10 часов sibkron RE:«Уроки русского» 20 часов babajga RE:сиреневая звезда 22 часа edvud RE:Не работает регистрация и восстановление пароля 2 дня нэнси RE:Подайте бедному копеечку на книжку с литреса... 3 дня Larisa_F RE:Грушевое дерево 4 дня koifish RE:Оплатил абонемент, деньги списались, абонемента нет 4 дня Саша из Киева RE:Неудавшийся священник 6 дней babajga RE:Повесть о чудесном одуванчике 1 неделя Саша из Киева RE:Ночной пассажир 2 недели larin RE:Оплатила,но абонемент не отображается 2 недели tvnic RE:Maxima-library - новый адрес 2 недели Саша из Киева RE:Хождение во власть. После путча 2 недели babajga RE:Алиса в стране чудес 3 недели babajga RE:Ёжик, который хотел обнять Луну 3 недели babajga RE:Самый храбрый совёнок 3 недели babajga RE:Похититель домофонов 3 недели Впечатления о книгах
Dongel про Горький: Дед Архип и Лёнька (Русская классическая проза, Детская проза)
11 05 <<Прочитав в соответствующем -- т.е. в младшем школьном возрасте -- этот рассказ, я искренне, от души, навсегда возненавидел Максима Горького.>> Да, оставляет ощущение полной безнадеги. И я даже затрудняюсь с оценкой. ……… Оценка: отлично!
Дей про Хонихоев: Сяо Тай, специалист по переговорам [СИ] (Юмористическая фантастика, Фэнтези, Самиздат, сетевая литература)
11 05 Автор что, не помнит, о чём писал страницу тому назад? сначала его герой мальчик, потом внезапно девочка.
udrees про Михайлов: Запертый [СИ] (Боевая фантастика, Постапокалипсис, Самиздат, сетевая литература)
11 05 Эта книга мне больше понравилась чем серия книг про Низшего гоблина Оди. Главный герой тут тоже низший и униженный, но пока в первой книге не такой безжалостный убийца и садист как Оди. Он тоже правдоруб, борец за справедливость, ……… Оценка: отлично!
udrees про Михайлов: Инфер-2 [СИ] (Боевая фантастика, Самиздат, сетевая литература)
11 05 Достойное продолжение для любителей серии про Низшего Оди. Описание все такое же стандартное, главный герой, не знаю почему, у меня не вызывает никакой симпатии. С одной стороны это властный, безжалостный и авторитарный лидер, ……… Оценка: хорошо
tamuh2 про Геммы
10 05 Классная книга, жду продолжения, замечательный мир с геммами, читала с удовольствием.
miri.ness_ про Еще не поздно (СИ)
09 05 Автор эрудирован (как специалист по МЭ говорю, из Зеленограда), но патологически глуп. Он так быстро накропал пять романов, что понятно, что уничижительные слова для всего всего, что было в жизни его родителей, он не подбирал ………
Sello про Труайя: Алеша (Историческая проза, Биографии и Мемуары)
09 05 О самой повестушке особо нечего сказать. Добротный язык, без выкрутасов, ясное содержание - пишет Труайя хорошо, что тут наворачивать словеса ради слов. Не первое его произведение, прочитанное мной. Другое приходит на ум: ……… Оценка: хорошо
decim про Карнеро: Пазолини. Умереть за идеи [litres] (Биографии и Мемуары, Изобразительное искусство, фотография)
09 05 Возбуждающая начальство тематика вымарана цензурой до такой степени, что от книги остались обрывки. Нечитаемо. Оценка: нечитаемо
miri.ness_ про Шестаков: Монгольское нашествие на Русь и Европу [litres] (История)
07 05 Что интересно. Вчера посетил итальянскую педию (перешёл с русской), по слову Тамерлан (Тимур). Там употребляется выражение "тюрко-монгольская орда", татар и в помине нет. Оценка: неплохо
Олег Макаров. про Попытка возврата
07 05 Написано интересно, качественно. Всё смазывается тем, что автор либо ненавидит, либо презирает всех, кроме Сталина и русских. * Ещё забавляют некоторые моменты неграмотности автора, типа французкий луковый суп это ………
Sello про Замятин: Мы (Социальная фантастика, Антисоветская литература)
07 05 Вчера состоялся давно с нетерпением ожидавшийся всеми День Единогласия. В 48-й раз единогласно избран все тот же, многократно доказавший свою непоколебимую мудрость Благодетель. Торжество омрачено было некоторым замешательством, ……… Оценка: хорошо |
Комментарии
Текст изменения в author.inc, прошу знающих php глянуть
Текст предлагаемого изменения в author.inc, прошу знающих php глянуть на предмет косяков.
Old (file author.inc, function AddFileToLibrusec lines 609-610):
if ($dublid = Sel ("BookId FROM libbook JOIN libavtor USING (BookId) WHERE Title="$title" AND AvtorId = $a AND BookID != $b AND NOT Deleted"))
$r .= "<p>Кажется, нашли дубля. Ткните в него для разрешения конфликта. <a href=/b/$b/join/$dublid target=_blank>Дубль: $title</a>";
New (вариант при котором автоматическая замена работает для всех):
if ($dublid = Sel ("BookId FROM libbook JOIN libavtor USING (BookId) WHERE Title="$title" AND AvtorId = $a AND BookID != $b AND NOT Deleted"))
{
$r .= "<p>Кажется, нашли дубля.";
//
// $b - new book Librusec-book-number, not the FICTIONBOOK/DESCRIPTION/DOCUMENT-INFO/ID
// $dublid - dubl Librusec-book-number
// $id - new book document-info-ID (was set earlier)
// $ver - new book document-info-version (was set earlier)
//
// Get document-info-ID and version for dubl
$dubldocid = Sel ("Id FROM libbook WHERE BookId = $dublid"); // ya ne uveren v sintaksise
$dublver = Sel ("Ver FROM libbook WHERE BookId = $dublid");
// Automatic upgrade of the same book
if( ($dubldocid==$id) && (1.0*$ver > 1.0*$dublver) ){
$r .= joinbooks($b,$dublid,$title);
return $r . "<hr>\n";
};
// Automatic upgrade of the old "LibRusEc kit" book
// 012345678901234567890123
// U libruseka dva tipa id - "Tue Jun 12 03:34:15 2007" i "Mon Jun 10 19:57:46 2013"
// Kljuchevoe - probely s godom.
// 012345678901234567890123
// ("ID"="... ... .. ..:..:.. 2013" OR "ID"="... ... .. ..:..:.. 2007")
$i = strrpos($dubldocid," 20");
if( ($i==19) && (1.0*$ver > 1.0*$dublver) ){
$r .= joinbooks($b,$dublid,$title);
return $r . "<hr>\n";
};
// No automatic resolution, resolve conflict manually
$r .= " Ткните в него для разрешения конфликта. <a href=/b/$b/join/$dublid target=_blank>Дубль: $title</a>";
};
New (вариант при котором автоматическая замена работает только для библиотекарей):
if ($dublid = Sel ("BookId FROM libbook JOIN libavtor USING (BookId) WHERE Title="$title" AND AvtorId = $a AND BookID != $b AND NOT Deleted"))
{
$r .= "<p>Кажется, нашли дубля.";
if (user_access('библиотекарь'))
{
//
// $b - new book Librusec-book-number, not the FICTIONBOOK/DESCRIPTION/DOCUMENT-INFO/ID
// $dublid - dubl Librusec-book-number
// $id - new book document-info-ID (was set earlier)
// $ver - new book document-info-version (was set earlier)
//
// Get document-info-ID and version for dubl
$dubldocid = Sel ("Id FROM libbook WHERE BookId = $dublid"); // ya ne uveren v sintaksise
$dublver = Sel ("Ver FROM libbook WHERE BookId = $dublid");
// Automatic upgrade of the same book
if( ($dubldocid==$id) && (1.0*$ver > 1.0*$dublver) ){
$r .= joinbooks($b,$dublid,$title);
return $r . "<hr>\n";
};
// Automatic upgrade of the old "LibRusEc kit" book
// 012345678901234567890123
// U libruseka dva tipa id - "Tue Jun 12 03:34:15 2007" i "Mon Jun 10 19:57:46 2013"
// Kljuchevoe - probely s godom.
// 012345678901234567890123
// ("ID"="... ... .. ..:..:.. 2013" OR "ID"="... ... .. ..:..:.. 2007")
$i = strrpos($dubldocid," 20");
if( ($i==19) && (1.0*$ver > 1.0*$dublver) ){
$r .= joinbooks($b,$dublid,$title);
return $r . "<hr>\n";
};
}; // if библиотекарь
// No automatic resolution, resolve conflict manually
$r .= " Ткните в него для разрешения конфликта. <a href=/b/$b/join/$dublid target=_blank>Дубль: $title</a>";
};
Отв: Как нам обустроить заливку KIT-4063
Да, до завтра жду замечаний-предложений, потом прошу Ларина внести изменение, и если он согласится - то кто не спрятался, я не виноват ;-)
Отв: Как нам обустроить заливку KIT-4063
Честно признаю, что я чайник. Поэтому и вопрос у меня скорее всего дурацкий. Но если можно я его задам. Не открывает ли это возможность для вредительства. Предположим кто-то берёт имеющийся у нас файл, текст в нём заменяет на некую абракадабру и затем заливает на ЛибРусЭк как апргрейженый. Автоматически старый файл с полным текстом заменяется на абракадабру. А если такое произойдёт в массовом порядке?
Отв: Как нам обустроить заливку KIT-4063
Вопрос точный и правильный. Я себе это так представляю: для того, чтобы проделать такую диверсию, нужно сделать много-много-кликов-на-одну-книгу. А откатывается это дело - одним кликом. Обнаружение и вовсе не представляет проблемы - новинки пасут почитай что все. Поэтому не думаю, что тут откроется много возможностей для вредительства.
Отв: Как нам обустроить заливку KIT-4063
Я тоже чайник в том смысле, что не разбираюсь в скриптах. Поэтому только замечу, что если скрипт будет изменен, надо сначала залить несколько книг и тщательно проверить, что получилось, а только потом заливать все. Насчет злоупотреблений и вредительства - это тоже возможно, при автоматическом апдейте можно представить себе бота, который будет скачивать книги, менять текст в них любым способом, и заливать обратно "исправленные". От этого есть очень простое средство: массвоые заливки / апдейты с использованием автоматики сделать доступными только библиотекарям, а для простых смертных оставить все, как есть. Можно также просто поставить капча-фильтр, чтобы авторизация и скачка книг проходила как обычно, а вот заливка, переименование, изменение жанра и прочие креативные операции - только после набора кода с картинки. Это не сильно усложнит операции по заливке, но зато увеличит уровень защищенности. Для библиотекарей, естественно, такой защиты не надо.
Отв: Как нам обустроить заливку KIT-4063
Я, честно говоря, не думаю что там так все просто с ботами. Но на всякий случай добавил ещё вариант, при котором автоматическая замена работает только для библиотекарей.
Отв: Как нам обустроить заливку KIT-4063
Заметка для отметки. Просьба Ларину внести изменение отправлена.
Отв: Как нам обустроить заливку KIT-4063
Внёс.
Отв: Как нам обустроить заливку KIT-4063
Спасибо.
И сразу проблемка :((
Заливка первых 20-ти книг на пробу показала, что первый "if"
// Automatic upgrade of the same book
if( ($dubldocid==$id) && (1.0*$ver > 1.0*$dublver) ){
$r .= joinbooks($b,$dublid,$title);
return $r . "<hr>\n";
};
срабатывает штатно.
А второй "if"
// Automatic upgrade of the old "LibRusEc kit" book
// 012345678901234567890123
// U libruseka dva tipa id - "Tue Jun 12 03:34:15 2007" i "Mon Jun 10 19:57:46 2013"
// Kljuchevoe - probely s godom.
$i = strrpos($dubldocid," 20");
if( ($i==19) && (1.0*$ver > 1.0*$dublver) ){
$r .= joinbooks($b,$dublid,$title);
return $r . "<hr>\n";
};
- не срабатывает. joinbooks который у него внутре, никогда не происходит. То бишь условие if( ($i==19) && (1.0*$ver > 1.0*$dublver) ) не выполняется на тех книжках, где железно должно было бы выполняться... Например, пары книг (я пока держу их в незакрытых окнах браузера):
http://lib.rus.ec/b/120358/join/288
http://lib.rus.ec/b/120363/join/538
Я не знаю в чем причина... вроде по TFM strrpos всё должно работать :((