skaut / skautis-integration

WordPress plugin pro možnost přihlašování a registrace do WordPressu přes skautIS
https://cs.wordpress.org/plugins/skautis-integration/
GNU General Public License v3.0
3 stars 5 forks source link

Profilové fotky #43

Open kalich5 opened 6 years ago

kalich5 commented 6 years ago

Co do WordPressu tahat i profilové fotky, co si lidi do skautISu přidávají. Aby u profilu nějakou fotku měli...

davidulus commented 6 years ago

WordPress nativně podporuje profilové fotky jen přes Gravatar => nemá je lokálně uložené, zobrazují se z externího zdroje. To skautIS neumožňuje, profilové fotky se zobrazí jen uživatelům, kteří k tomu mají práva. Znamenalo by to tedy duplikovat funkcionalitu některého z hotových pluginů (https://cs.wordpress.org/plugins/wp-user-avatar/, https://cs.wordpress.org/plugins/custom-user-profile-photo/), které umožňují vlastní profilové fotky, které jsou uloženy ve WP. Na ně pak lze navázat automatické stažení profilové fotky ze skautISu, pokud uživatel ve WP ani Gravataru žádnou nemá, ale obávám se, že mezi těmito pluginy není jednoznačná preference a dělat to kompatibilní naráz s cca. 4 pluginy je nesmysl. Proto doporučuji od této funkce buď zcela upustit nebo se rozhodnout pro jeden plugin, který umožňuje vlastní profilové fotky a na něj navázat stažení fotografie ze skautISu.

jerrysohn commented 6 years ago

Nevím, jestli to jakkoli něco ovlivňuje, ale potřebuji upozornit, že skautIS umí vrátit profilou fotku i anonymně (tedy bez nutnosti ověření uživatele; stačí aby k dané funkci měla přístup daná aplikace - měla povolenou danou funkci). Využívá se to např. při zobrazení lektorů, kde si to ze skautISu načte fotky a ty nikde jinde než ve skautISu nejsou (viz např. https://krizovatka.skaut.cz/vzdelavaci-akce/416-ck-vocat-pocad-2017)

kalich5 commented 6 years ago

Jako to co psal Jerry jsem si myslel taky. Stačí to pro tu aplikaci povolit a fotky se stahovat budou. Jako mít to nativně v našem pluginu by bylo podle mě lepší, než být závislí na nějakém dalším. Nevím jak to může být složité implementovat. Na druhou stranu jsou třeba ty dva pluginy pod GPL 2 licencí, takže by mělo jít jejich částí normálně okopírovat na našeho pluginu.

davidulus commented 6 years ago

Duplikovat v našem pluginu funkcionalitu nějakého již hotové, znamená se o ten kód také následně starat. Co jsem se koukal, jde nejméně o 300 řádků kódu, což je pro představu podobné množství kódu, jako nyní slouží pro obsluhu shortcodes v našem pluginu. "mít to nativně v našem pluginu by bylo podle mě lepší, než být závislí na nějakém dalším" - jsem právě spíš toho názoru, že jde o přirozenou závislost => umíme tahat fotky ze skautISu do profilovek uživatelů, ale v závislosti na pluginu, který doplňuje do WordPressu funkci vlastních profilovek. Stejně jako neduplikujeme funkcionalitu třeba pluginů pro práci s vlastními rolemi a oprávněními - už je někdo vytvořil a stará se o ně. Není to ambicí tohoto pluginu a WordPress to nativně neumí. Jaké jsou důvody nebýt v tomto závislý na dalším pluginu?

kalich5 commented 6 years ago

No už jsem používal pár pluginu, které vývojář prostě přestal dělat a při velké aktualizaci WP se mi web rozsypal. Měl jsem šablonu, která byla na druhé závislá, vývojář tu hlavní přepsal a mě se to rozsypalo. Proto jsem pro to, nebýt na ničem závislý. Ale hlavní je asi tvoje rozhodnutí Davide, jak by se to to chtělo implementovat.

davidulus commented 6 years ago

Navrhuji vybrat plugin, na kterém to bude závislé a pro něj to dopsat jako rozšíření se skautISem. Při registraci nového uživatele by se profilová fotka nahrála ze skautISu. Např. https://wordpress.org/plugins/metronet-profile-picture/

kalich5 commented 6 years ago

Ok, závislost na pluginu není problém. Ten plugin na kterém to bude závislé nechám na tobě. Ale čím menší bude tím lépe. Ten co jsi poslal, vypadá v pohodě. Lepší než ty na začátku, přeci jen jsou dost velké.

Tak že bychom použili https://wordpress.org/plugins/metronet-profile-picture/

A otázka je... a) bude to vyžadováno vždy? b) bude to závislé na aktivování "Modulu" - Profilové fotky

Co myslíš?

davidulus commented 6 years ago

OK. Nevyžadoval bych to vždy. V nastavení Registrace bych dal zašedlé / disabled zatržítko - profilové fotky ze skautISu, u kterého bude poznámka, že je třeba nainstalovat daný plugin a po kliknutí by Tě to hned hodilo na instalaci toho pluginu. Po jeho nainstalování to již půjde zatrhnout. Může být? Texty doladíme posléze.

kalich5 commented 6 years ago

Tak by to mohlo být v pohodě.

A jak to bude na webech, kde je pouze přihlašování? Kde není registrace vůbec puštěná (uživatelé jsou už propojený)? Nebo je propojuje správce? Jak to bude když si ve skautISu fotku změním? Bude někde tlačítko aktualizovat fotku ze skautISu?

davidulus commented 6 years ago

To je dobrá myšlenka, díky => bude třeba to udělat jako samostatný modul a nenavazovat to na aktivovaný modul Registrace. Pokud uživatel nebude mít žádný gravatar nebo již vlastní nahranou fotku - natáhne se ze skautISu.

"Bude někde tlačítko aktualizovat fotku ze skautISu?" - dobrý nápad => bude

kalich5 commented 6 years ago

Tak jako i když bude mít třeba gravatar (bude chtít třeba fotku ze skautISu), a to by podle mě s tím pluginem co jsi posílal jít mělo.