<?php // Полная загрузка сервисных книжек, создан 2024-01-05 12:44:55
global $wpdb2;
global $failure;
global $file_hist;
///// echo '<H2><b>Старт загрузки</b></H2><br>';
$failure=FALSE;
//подключаемся к базе
$wpdb2 = include_once 'connection.php'; ; // подключаемся к MySQL
// если не удалось подключиться, и нужно оборвать PHP с сообщением об этой ошибке
if (!empty($wpdb2->error))
{
///// echo '<H2><b>Ошибка подключения к БД, завершение.</b></H2><br>';
$failure=TRUE;
wp_die( $wpdb2->error );
}
$m_size_file=0;
$m_mtime_file=0;
$m_comment='';
/////проверка существования файлов выгрузки из 1С
////файл выгрузки сервисных книжек
$file_hist = ABSPATH.'/_1c_alfa_exchange/AA_hist.csv';
if (!file_exists($file_hist))
{
///// echo '<H2><b>Файл обмена с сервисными книжками не существует.</b></H2><br>';
$m_comment='Файл обмена с сервисными книжками не существует';
$failure=TRUE;
}
/////инициируем таблицу лога
/////если не существует файла то возврат и ничего не делаем
if ($failure){
///включает защиту от SQL инъекций и данные можно передавать как есть, например: $_GET['foo']
///// echo '<H2><b>Попытка вставить запись в лог таблицу</b></H2><br>';
$insert_fail_zapros=$wpdb2->insert('vin_logs', array('time_stamp'=>time(),'last_mtime_upload'=>$m_mtime_file,'last_size_upload'=>$m_size_file,'comment'=>$m_comment));
wp_die();
///// echo '<H2><b>Возврат в начало.</b></H2><br>';
return $failure;
}
/////проверка лога загрузки, что бы не загружать тоже самое
$masiv_data_file=stat($file_hist); ////передаем в массив свойство файла
$m_size_file=$masiv_data_file[7]; ////получаем размер файла
$m_mtime_file=$masiv_data_file[9]; ////получаем дату модификации файла
////создаем запрос на получение последней удачной загрузки
////выбираем по штампу времени создания (редактирования) файла загрузки AA_hist.csv, $m_mtime_file
///// echo '<H2><b>Размер файла: '.$m_size_file.'</b></H2><br>';
///// echo '<H2><b>Штамп времени файла: '.$m_mtime_file.'</b></H2><br>';
///// echo '<H2><b>Формирование запроса на выборку из лога</b></H2><br>';
////препарируем запрос
$text_zaprosa=$wpdb2->prepare("SELECT * FROM `vin_logs` WHERE `last_mtime_upload` = %s", $m_mtime_file);
$results=$wpdb2->get_results($text_zaprosa);
if ($results)
{ foreach ( $results as $r)
{
////если штамп времени и размер файла совпадают, возврат
if (($r->last_mtime_upload==$m_mtime_file) && ($r->last_size_upload==$m_size_file))
{////echo '<H2><b>Возврат в начало, т.к. найдена запись в логе.</b></H2><br>';
$insert_fail_zapros=$wpdb2->insert('vin_logs', array('time_stamp'=>time(),'last_mtime_upload'=>$m_mtime_file,'last_size_upload'=>$m_size_file,'comment'=>'Загрузка отменена, новых данных нет, т.к. найдена запись в логе.'));
wp_die();
return $failure;
}
}
}
////если данные новые, пишем в лог запись о начале загрузки
/////echo '<H2><b>Попытка вставить запись о начале загрузки в лог таблицу</b></H2><br>';
$insert_fail_zapros=$wpdb2->insert('vin_logs', array('time_stamp'=>time(),'last_mtime_upload'=>0, 'last_size_upload'=>$m_size_file, 'comment'=>'Начало загрузки'));
////очищаем таблицу
$clear_tbl_zap=$wpdb2->prepare("TRUNCATE TABLE %s", 'vin_history');
$clear_tbl_zap_repl=str_replace("'","`",$clear_tbl_zap);
$results=$wpdb2->query($clear_tbl_zap_repl);
///// echo '<H2><b>Очистка таблицы сервисных книжек</b></H2><br>';
if (empty($results))
{
///// echo '<H2><b>Ошибка очистки таблицы книжек, завершение.</b></H2><br>';
//// если очистка не удалась, возврат
$failure=TRUE;
wp_die();
return $failure;
}
////загружаем данные
$table='vin_history'; // Имя таблицы для импорта
//$file_hist Имя CSV файла, откуда берется информация // (путь от корня web-сервера)
$delim=';'; // Разделитель полей в CSV файле
$enclosed='"'; // Кавычки для содержимого полей
$escaped='\
1. Запрос медленный, т.к. отбор составлен неверно. Отбирать нужно в условиях для виртуальной таблицы.
2. Текст запроса ИМХО скопирован из какого-то отчета, т.к. в тексте полно секций с фигурными скобками { и }, которые в обычном запросе не используются.
И т.д. и т.п.
Исправляйся.
Тогда поставлю плюс.
(0)
3. также в запросе запрашивается куча ненужных полей
4. периодичность по регистратору также не нужна — убери запрос будет побыстрее.
5. Было бы удобно добавить заполнение с ограничением по физ.лицу.
Спасибо за комментарий! Запрос изменила, но у нас скорость запроса по старому =8, а по новому 6 сек на 450 строках. Это нормально.
Да, действительно запрос взят из какого-то отчета, но просто надо срочно, ведь конец года.
Заполнение с ограничением по физ.лицу не надо, тк документ списания оформляется
по подразделению, хотя уточню нужность у бухов. Запрос до конца упрощу позже.
Еще раз спасибо за науку!
Выложена последняя версия с упрощенным запросом. А где обещанный плюс?
(0) 6. Лучше юзать ВТ Остатки вместо ВТ ОстаткиИОбороты
Почему? Информация достоверна в обоих.
7. За такой код
Номенклатура.Загрузить(Запрос.Выполнить().Выгрузить()); Номенклатура.Свернуть("ФизЛицо,Номенклатура,НазначениеИспользования,ДокументПередачи", "КолПриход,КолРасход,СумПриход,СумРасход,ПогашСумПриход,ПогашСумРасход"); Для Каждого ТекущаяСтрока Из Номенклатура Цикл ТекущаяСтрока.Остаток = ТекущаяСтрока.СумПриход - ТекущаяСтрока.ПогашСумПриход; ТекущаяСтрока.Дата= ТекущаяСтрока.ДокументПередачи.Дата; ТекущаяСтрока.Срок=ТекущаяСтрока.НазначениеИспользования.СрокПолезногоИспользования; КонецЦикла;Показать полностью
Плюс ты никак не получишь 🙁
Весь этот код прекрасно выполняется в запросе — т.е. выше скорость выполнения, код проще и т.д.
Например, я только что сделал аналогичную обработку по заполнению ТЧ для списания спец.одежды по организации, подразделению, физ.лицу.
Там все просто 🙂 :
Показать полностью
(7) Тебе же нужны только остатки на нужную дату, движения/обороты ты зря юзаешь 🙁 ИМХО это неверно.
Хорошо, убедил. На следующей неделе исправлю. Но, по-моему, ты теоретик, что-то все выкапываешь, а я практик, на мне кроме этого весь УПП висит,
от З/П до себестоимости. Критику восприняла. Исправлюсь…, но все же плюсани!
Хотя бы за то, что прислушиваюсь, и принимаю к действию…
(10) Без теории нормальной практики не будет 🙂
Если ты сделаешь нормальный вариант, и самой будет полезно, и людям приятно 🙂
На мне самом висит и УПП, и ЗУП-ы разных контор, + куча различных семерочных баз и т.д. и т.п.
Не обижайся, твой опыт только повысится от этого, в следующий раз другую вещь лучше и быстрее сделаешь.
Жду исправлений, а уж потом будет плюс 🙂
Исправила запрос по заполнению, но в печатной форме бухи хотят видеть и приход и расход, так что оставила. Реагируй!
(0) Также в описании нужно отметить, что обработка рассчитана на очень специфически способ ведения учета 🙁 — одно списание за период для одного подразделения, т.е. не очень универсальна, к сожалению.
(12) Бухгалтеров сильно не слушай, они очень часто сами не знают, что хотят 🙁
Зачем им в данной обработке все движения за всю жизнь базы?
Для этого существуют штатные специализированные отчеты по спецодежде, например, по партиям.
В итоге сейчас тратится куча времени на получение движений 🙁
Убери в запросе периодичность по регистратору, жить станет веселее 🙂
Исправила. А где плюс?!!!
(12) Твоя обработка предназначена только для списания, поэтому тут никакие приходы/расходы не нужны, а только остатки на текущий/нужный момент.
Юзай ВТ Остатки.
ЗЫ все равно отчет по партиям материалов в эксплуатации мощнее твоей обработки, за ним не нужно гнаться — отборы, группировки и т.п. настройки там лучше.
Не каждый пользователь может настроить так как ему надо, и потом, здесь есть АВТОМАТИЧЕСКОЕ формирование всего…
(15) Цитата: «Одно списание за период для одного подразделения, но можно наложить фильтр по человеку и списать по нему.»
В коде не увидел ничего подобного по физ.лицу 🙁
Тестирование также показало, что отбор по физ.лицу не действует 🙁
Далее — у тебя практически одинаковые куски кода по УУ и БУ, убери это дублирование, и код станет проще и надежнее.
Также поправь код
[code]
|ИЗ
| РегистрНакопления.ПартииМатериаловВЭксплуатации.Остатки(
| &ДатаКон,
| Подразделение = &Подразделение
| И ДокументПередачи.Проведен = &Проведен) КАК ПартииМатериаловВЭксплуатацииОстатки
|ГДЕ
| ПартииМатериаловВЭксплуатацииОстатки.ДокументПередачи.Проведен = &Проведен
[code]
1. Одна проверка на Проведен лишняя 🙂
2. Если нужен только непроведенный док, можно написать проще — НЕ ДокументПередачи.Проведен — т.е. без лишнего параметра
(17) Пользователя очень-очень редко нужно все 🙁
Большие объемы данных все равно никто никогда не будет разбирать, это давным-давно известно.
В УПП в типовых отчетах есть возможность импортировать чужие настройки как свои прямо в форме отчета, очень удобная фича как раз для разных пользователей.
«Тестирование также показало, что отбор по физ.лицу не действует :(»
все работает, а то как бы мы могли проверять по бешеному кол-ву людей!!!
«В коде не увидел ничего подобного по физ.лицу :(» а это и не в коде, а в фильтре на ТЧ
И, вообще это выискивание блох…
А двойное условие — из-за того, что торопилась.Sorry!
(20) Еще небольшая подсказка:
Вместо кода типа
Показать полностью
можно, а чаще всего нужно, написать проще и быстрее:
Показать полностью
Т.е. все одинаковые свойства 1С сама быстро занесет в НовСтрока, а ты только те, у которых наименования расходятся 🙂
За подарок и ликбез спасибо.
(20) По отбору по физ.лицу
у тебя самой написано: «можно наложить фильтр по человеку и списать по нему.»
Я же говорю именно о списании по физ.лицу, а не о отборе в табличной части 🙂
Попробуй наложи этот фильтр и сделай так, чтобы документ создался только по этому человеку.
А ведь этого не получится в текущем варианте обработки — ИМХО! Если не прав, жду комментария 🙂
ЗЫ да и сама подумай, если у тебя возможен всего один док на подразделение, значит, списание на одного чела не имеет смысла, т.к. списание на след.чела удалит пред.документ 🙁
У нас задача списания по человеку не стояла. Надо было списать по подразделению,
а фильтр для удобства работы. Обойдут всех челов, а потом только жмут на сформировать документ. Согласна, что можно доработать — ведь нет предела совершенству…
Выложила, что получилось( удобное и работающее у меня), код открыт, кому надо тот
доделает!
То что мне надо, только почему то списывает одно и тоже каждый месяц, я списал ботинки уже с человека, а он на следующий месяц опять их же списывает. Сейчас буду искать и править ошибку.
Видимо ошибка в комплексной конфе, документ списание материалов из эксплуатации не влияет на регистр накопления «Партии материалов в эксплуатации». Но влияет на регистр «материалы в эксплуатации»