[documentview] podglad pdf - pusta strona about lms

[voytec69]

tak wygląda w bazie dokument którego nie mogę wyświetlić

SELECT * from documentattachments  WHERE docid = 317160 ORDER BY docid ASC, type DESC;
  id   | docid  |         filename         |   contenttype   |              md5sum              | type | cdate
-------+--------+--------------------------+-----------------+----------------------------------+------+-------
 29220 | 317160 | ssssss_DANUTA - u.pdf | application/pdf | 94197cc3186fb7b8b2f7ffd644ca7b8f |    0 |     0

[voytec69]

Zapytania z procesu wyświetlania teg dokumentu wyglądają tak.

SELECT 1 FROM documents WHERE id = '317160';
 ?column?
----------
        1
(1 row)

SELECT d.id FROM documents d
                JOIN docrights r ON (r.doctype = d.type)
                WHERE d.id IN (317160)
                    AND d.type < 0
                    AND r.userid = '2'
                    AND (r.rights & 1) > 0;
   id
--------
 317160
(1 row)

SELECT filename, contenttype, md5sum
                    FROM documentattachments
                    WHERE docid IN ('317160') AND type = 1 ORDER BY docid ASC, type DESC;
 filename | contenttype | md5sum
----------+-------------+--------
(0 rows)

[voytec69]

Commit 6527b4f

Było

lms/modules/documentview.php

Lines 52 to 53 in 578dbe3

	WHERE docid IN ?
	ORDER BY docid ASC, type DESC',

Jest

lms/modules/documentview.php

Lines 66 to 69 in 6527b4f

	WHERE docid IN ?'
	. ($attachments || !empty($attachmentid) ? '' : ' AND type = 1')
	. (empty($attachmentid) ? '' : ' AND id = ' . $attachmentid)
	. ' ORDER BY docid ASC, type DESC',

Zmieniłem linie u siebie

lms/modules/documentview.php

Line 67 in 6527b4f

. ($attachments || !empty($attachmentid) ? '' : ' AND type = 1')

na

. ($attachments || !empty($attachmentid) ? '' : '')

I działa, ale to słabe rozwiązanie problemu :) i zapewne wygeneruje inne...

[voytec69]

O ile dobrze rozumiem logikę documentadd.php to plik generowany w systemie ma

lms/modules/documentadd.php

Line 235 in fa49e3e

'attachmenttype' => 1,

a ten uploadowany ma

lms/modules/documentadd.php

Line 252 in fa49e3e

$attachment['attachmenttype'] = 0;

jak to się ma do tego warunku type = 1 w documentview.php ?

[chilek]

attachmenttype = 1 (pole type w tabeli documentattachments) - to załącznik główny dokumentu i jest zwykle generowany na podstawie szablonu HTML.

[chilek]

Jest

lms/modules/documentview.php

Lines 66 to 69 in 6527b4f

	WHERE docid IN ?'
	. ($attachments || !empty($attachmentid) ? '' : ' AND type = 1')
	. (empty($attachmentid) ? '' : ' AND id = ' . $attachmentid)
	. ' ORDER BY docid ASC, type DESC',

Jeśli do ?m=documentview przekazano attachmentid, tzn. że chcemy obejrzeć konkretny plik z załącznikem dokumentu i wtedy potrzebny jest warunek AND id = ' . $attachmentid, a jednocześnie nie powinniśmy testować, czy załącznik jest głównym (niepotrzebny więc warunek AND type = 1).

Jeśli $attachments jest niepuste, tzn. że chcemy obejrzeć wszystkie załączniki w postaci sklejonego HTML lub PDF i dlatego też nie powinniśmy tworzyć warunku AND type = 1).

Typ własny dokumentu nie ma nic wspólnego z typem załącznika. Typ dokumentu to pole type w tabeli documents, a typ załącznika dokumentu to pole type w tabeli documentattachments.

Z ciekawości - co pokazuje Ci polecenie:

pdfinfo 'ssssss_DANUTA - u.pdf'

?

[chilek]

Jaki wynik zwraca zapytanie:

SELECT filename, contenttype, md5sum
                    FROM documentattachments
                    WHERE docid = 317160;

?

[voytec69]

Wynik już wrzucałem wyżej:
#2330 (comment)

i tu proces z logow postgresa. Jak widac dla tego dokumentu testowal warunek and type = 1 w konsekwencji nie zwraca tego co trzeba.
#2330 (comment)

Ta logika którą wprowadziłeś, za pewne nie ogarnia (bo skąd), sposobu dodawania pdfów (zaszłość) przez bok.
Odbywa się to poprzez Dokument->Nowy dokument.

attachmenttype = 1 (pole type w tabeli documentattachments) - to załącznik główny dokumentu i jest zwykle generowany na podstawie szablonu HTML.

No czyli tu pewnie jest pies pogrzebany, bo dokument nie powstał na bazie żadnego szablonu.

Typ dokumentu skan jest własny, dodany w definitions.php

Nie napisałem precyzyjnie też, bo to dotyczy wszystkich dokumentów dodanych w ten sposób, ten jeden to przykład tylko.
(do momentu wprowadzenia tego warunku, otwierały się wszystkie bez problemu).

[voytec69]

Żeby była jasność, to te pdfy to fizyczne skany nie wiem, ulotki, oświadczenia, mapki ale też samej umowy wydrukowanej i podpisanej przez klienta.

[voytec69]

odwzorowałem to na demo.lms.org.pl https://demo.lms.org.pl/?m=documentlist&c=10 dokument aaa_test

[voytec69]

@chilek potrzebne ci dodatkowe info?

[chilek]

Proszę o testy po poprawce, którą przesłałem.

[voytec69]

To samo.
PHP Warning: Invalid argument supplied for foreach() in /modules/documentview.php on line 76

Problem jest raczej w zapytaniu które powinno coś zwrócić.

[voytec69]

Poprawka, jednak jest ok, mialem jedno niezamknięte okno LMSa, które wprowadziło mnie w błąd.

[voytec69]

Dzięki, zamykam.

[voytec69]

Jednak jak klika się w wiersz to się nie otwiera dokument. testowane na demo.lms.org.pl

blank_page_docs_s2.mov

[interduo]

Może warto użyć w tym miejscu:
https://github.com/interduo/lms/blob/master/js/lms-ui-fileview.js ?

Fajnie to działa dla plików PDF.

[chilek]

Naprawione.