код в Queryclose

[oxystile]

Доброго!

такая вот непонятка
в QueryClose код:

[codebox]t1=doc.ish_count(0) 'чисто для дебаггера
t2=dc.count 'для дебаггера

If doc.ish_count(0)<>dc.count Then
doc.ish_count=dc.count
Call ws.ViewRefresh
Call doc.Save(False,False)
end if[/codebox]

пусть, например t1<>t2 , при закрытии код выполнился (что видно в дебагере)
открываю док, значание переменной не изменилось, а при закрытии снова выполняется этот код
снова открываю док. и наконенц-то вижу, что значение изменено.

поле было и компутед и эдит, ни к чему оно не привязано, собственными глазами вижу, как оно получило новое значение (в дебагере) открываю, и вот нет его, этого значения...

 

[Omh]

Что в QC делает Call doc.Save(False,False)?

If doc.ish_count(0)<>dc.count Then Call doc.replaceItemValue("ish_count", dc.count)

Всё, не надо лишнего городить...

 

[Medevic]

А что за doc? Случаем не профильный документ?

Также, ты используешь False в параметрах: doc.Save(False,False). Неплохо бы посмотреть, что возвращает этот метод.
If Not doc.Save(False, False) Then MsgBox "Сохранение не удалось".

 

[Omh]

Хм, у меня даже не родилось мысли, что doc - это не текущий док, у которого QC и отрабатывает.
Заштампованость мышления

 

[oxystile]

Omh
doc это Source.Document
If doc.ish_count(0)<>dc.count Then Call doc.replaceItemValue("ish_count", dc.count) не работает, если save не выполнить, сколько не переоткрывай...
да и почему должен работать???

Medevic
может, я ошибаюсь в правильности наличия doc.Save(False, False) но
как раз в тех случаях, когда значение переменной и не обновлялось вставка этой проверки
If Not doc.Save(False, False) Then MsgBox "Сохранение не удалось".
и помогло определить что беда с сохранением
а почему, это уже надо искать

 

[Omh]

да и почему должен работать???

Потому что QueryClose, т.е. перед сохранением.
А зачем перед сохранением сохранять ещё раз?

 

[Medevic]

oxystile
В хелпе написано почему. Кто-то уже редактирует этот документ.

Потому что QueryClose, т.е. перед сохранением.
А зачем перед сохранением сохранять ещё раз?

Это не QuerySave.

 

[Omh]

Medevic
Смотрю в книгу, вижу фигу.
Вообще голова уехала.
Прошу прощения.

Сохрание документа на QueryClose зачастую свидетельствует о неправильной реализации.
Лучше избегать этого.

oxystile
Почему не перенести все проверки на QuerySave таки?

 

[oxystile]

вот то-то и я смотрю, что сперва QuerySave а потом QueryClose

но в QuerySave код не стоит запихивать, потому что не всегда этот метод будет вызван(если конечно не заставить)
это мне все-таки надо что-нибудь придумать на тему https://codeby.net/threads/22589.html

т.к. мне надо поставить признак наличия во входящем исходящих(которые оформлены в виде embedded view)
и учитывая то, что исходящие создаются только из входящего и т.о. QuerySave не учавствует во входящем, а QueryClose задействован по-любому, вот и добавила на форму поле ish_count, которое вычисляется при закрытии входящего и равно количеству исходящих (хотя можно установить его признаком, либо исх. есть либо нет)
я не претендую, что это идеальное решение, но как-то вот родилось...
так что если есть другие решения, приму с удовольствием

 

[Sandr]

Используйте метод сохранения уидокумента... Тогда проблем не будет..

 

[Omh]

Сделай глобальную переменную, в которую при входе зачитывай кол-во залинкованых доков.
При добавлении нового залинкованого дока, апдейт переменную.

Если так хочеться на сохранять док на QC, то именно там и проверяй соответствие переменной и текущего кол-ва залинкованых доков.
Если не равны, апдейть поле и сохраняй.

Я бы кампутед поле сунул бы в печЪ.

 

[Medevic]

так что если есть другие решения, приму с удовольствием smile.gif

Как вариант используем дополнительный документ. Назовем его счетчик.
1) Счетчик создается (как дочерний документ) при создании входящего документа.
2) В счетчик копируется необходимая часть данных из входящего документа, которая обновляется при сохранении входящего документа.
3) В представлении вместо входящего документа показываем счетчик с необходимыми данными из входящего документа. Соответственно, переопределяется событие Queryopendocument в представлении, чтобы открывался не счетчик, а входящий документ.

Да, и самое главное.
4) Все изменения исходящих документов пишем в счетчик агентом.

 

[oxystile]

Medevic
счетчик-документ, можно, но структура БД уже есть и на ней работают
и придется реализовывать п.1 на уже существующих документах...
+ такова уж практика, что все примочки кот. я добавляю в БД должны быть и легко "сносимы" (из-за 7 пятниц)
ну и прикручивать их надо ооочень аккуратненько, как бы так сказать сбоку

Sandr
почему именно метод сохранения уидокумента, а не doc.Save
doc.Save быстрее, а в querySave у меня набор всевозможных проверок, который необходим, если чел. действительно редактировал документ т.е. было и EditMode и он нажал сохранить
а такое действие как создание исходящего не нуждается потом в querySave входящего, т.к. все проверки неактуальны
да и к тому же этот уидок.save некуда и вставить (док открывается в реж. просмотра, из него добавляем исх., реж. просмотра при этом остается)


Omh
а почему поле компутед в печь? пусть будет edit
при входе зачитывать будет не совсем рационально
1. если документ свежак, то там будет 0 исходящих
2. чел открыл вход. и добавил исходящий, потом закрыл (и как рез-т надо чтобы соответсвующий icon появился в общем представлении напротив этого входящего)
а о том, нужен этот icon или нет, документ входящий узнает только при своем закрытии

на QueryClose у меня теперь код
t1=dc.count
If t1>0 And Cstr(doc.ish_flag(0))<>"1" Then
Call doc.replaceItemValue("ish_flag", 1)
End If
If t1=0 And Cstr(doc.ish_flag(0))="1" Then
Call doc.replaceItemValue("ish_flag", 0)
End If
If Not doc.Save(False,False) Then
Msgbox "Сохранение не удалось"
End If
Call ws.ViewRefresh

т.о. изменение флага на 1 будет только при добавлении первого исходящего, а дальнейшее их добавление не интересует.
ну и есть 1% случаев, когда исходящий (исходящие) будут удалены напрочь, и этот случай тоже отработает

 

[Sandr]

тобишь сохранение вы делаете и при ридмоде? А если пользователь не имеет право его сохранять? Используйте для счетчика служебные доки.. не придумывайте велосипед...

 

[oxystile]

пользователь имеет право сохранять, а даже если и включить ему доступ только на чтение, то ничего и не сломается, т.к. создать исходящий он ведь тоже не сможет, а следовательно и увеличить счетчик, где бы он не был

 

[Omh]

Не, я бы служебные доки не делал, а хранил бы метку.
Не вижу смысла городить ещё целую пачку доков, если можно обойтись изменеием поля.
Правда, в проблему въезжать уже сил нет, проще сказать, как с нуля всё построить

 

[oxystile]

Omh
ладно, вообщем, я поробую внедрить, если что и грохнется, то исправить легко можно

спасибо всем!

 

[Constantin A Chervonenko]

Сохрание документа на QueryClose зачастую сведетельствует о неправильной реализации.

ВСЕГДА.. не зачастую

Почему не перенести все проверки на QuerySave таки?

Ага. А если требуется лишнее сохранение, совместите его с основным - в PostSave (если только у вас не 4.х.х h34r: )

 

[K-Fire]

ВСЕГДА.. не зачастую

Допустим есть такая задача:

Главный документ "Счет". Подчиненные доки: "строка счета". Отображаются в ембеддед вью в Счете. Больше нигде не видны и из других месте не редактируются.
Вы добавляете еще одну "строку", вбиваете сумму внутри него. Закрываете документ строки.
Хочется чтобы сумма всех строк обновилась в "Счете" как можно быстрее, желательно сразу. Счет может находиться в режиме чтения или редактирования.

Какие ваши действия? :angry:

 

[Medevic]

Какие ваши действия?

И какие? :angry: