Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Znemožnění tisku hudebnin na studovnách #120

Closed
LenkaKravecova opened this issue Jun 28, 2016 · 12 comments
Closed

Znemožnění tisku hudebnin na studovnách #120

LenkaKravecova opened this issue Jun 28, 2016 · 12 comments
Assignees

Comments

@LenkaKravecova
Copy link
Collaborator

LenkaKravecova commented Jun 28, 2016

Viz ceskaexpedice#406
Nám jde jen o chráněné hudebniny (sheetmusic)... Jak by to šlo udělat?

@jirikrepl
Copy link

jirikrepl commented Jul 4, 2016

Díky ceskaexpedice#406 máme tři nové akce: PDF_RESOURCE, SHOW_CLIENT_PRINT_MENU a SHOW_CLIENT_PDF_MENU.

Tyto akce chceme zakázat pouze u podmnožiny dokumentů, které jsou private a mají model sheetmusic.

U podmnožiny dokumentů private sheetmusic není možné tyto akce zakázat globálně (v Nastavení práv globáních akcí). Práva by se musely nastavit jednotlivě u každého dokumentu.

V Krameriu chybí podmínka, díky které se mohou tyto akce omezit na výše zmiňovanou podmnožinu dokumentů.

@jirikrepl
Copy link

Knihovna modelfilter.jar byla přidána do nové verze Krameria moravianlibrary/s2i-kramerius@aa0f1d0 (kterou nasadíme místo staré verze)

Na svém PC jsem spustil novou verzi Krameria s knihovnou modelfilter.jar a nastavil pravidla. Vše funguje.

@LenkaKravecova
Copy link
Collaborator Author

Toto musíme nastavit v nové instalaci.

@LenkaKravecova LenkaKravecova removed their assignment Oct 5, 2016
@leiblix
Copy link
Collaborator

leiblix commented Oct 11, 2016

Mergnul jsem změnu do s2i-kramerius, mohl by to prosím někdo otestovat a nasadit? :-)

Tzn.

  1. vybuildit znovu s2i-kramerius (z naší branch hotfixapi)
  2. otestovat, že je to ok
  3. pushnout to na hub.docker.com
  4. udělat pull na server a otočit to

@leiblix
Copy link
Collaborator

leiblix commented Nov 1, 2016

Verze s touto změnou je nasazená, zbývá to nastavit.

@MarosGrm
Copy link

MarosGrm commented Nov 9, 2016

Nové nastavenie je viac-menej funkčné, pri hudobninách typu private sa stále zobrazuje možnosť tisku, ale samotný tisk už nie je možný pretože po odkliknutí stránka vracia chybu: Access to the specified resource has been forbidden.
Bude ešte treba spraviť aby sa možnosť tisku ani nezobrazovala.

@leiblix
Copy link
Collaborator

leiblix commented Dec 5, 2016

V originálním issue se píše:

Pokud bude funkcionalita v poradku, pridame filter do standardni distribuce.

@kenji1322 Nevíš jestli tu funcionalitu skutečně zařadili do distribuce? Pokud ano, je na místě odstranit tento hack - jinak v tom bude zmatek.

@leiblix leiblix reopened this Dec 5, 2016
@MarosGrm
Copy link

MarosGrm commented Dec 6, 2016

Neviem. A ono to vlastne nieje hack, práva sú nastavené tak ako sa spomína v originálnom issue pomocou pridanej funckionality. Na kramerius.mzk.cz to funguje správne, tak že pri private hudobninách sa možnosť tisku ani nezobrazuje. Problém je v tom, že na digitalniknihovna.cz sa táto možnosť zobrazí, ale po odkliknutí vracia vyššie spomínanú chybu.

@leiblix
Copy link
Collaborator

leiblix commented Dec 6, 2016

Omylem jsem nenalinkoval v předchozím postu tento řádek
https://github.com/moravianlibrary/s2i-kramerius/blob/master/Dockerfile#L55

Nelíbí se mi, že tam máme tu jar binárku, ke které jsou zdrojáky kdovíkde - může se to takto rozbít a možná by se to pak blbě opravovalo. Ideální by bylo, aby to začlenili do distribuce, jak bylo po otestování slíbeno.

@leiblix
Copy link
Collaborator

leiblix commented Dec 14, 2016

V nové verzi je to už součástí distribuce, je možné tu knihovnu z našeho s2i odstranit.

@MarosGrm
Copy link

Knihovna je odstranena moravianlibrary/s2i-kramerius@41cc533

@leiblix
Copy link
Collaborator

leiblix commented Feb 23, 2017

Díky.

@leiblix leiblix closed this as completed Feb 23, 2017
leiblix pushed a commit that referenced this issue Jun 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants