search

chamilo / chamilo-lms

Chamilo is a learning management system focused on ease of use and accessibility
https://chamilo.org
GNU General Public License v3.0
798 stars 480 forks source link

Error plugin Facebook #3158

Open mesoshqip opened 4 years ago

mesoshqip commented 4 years ago

Chamilo Version / Versión de Chamilo / Version de Chamilo

1.11.10

After activation of Facebook, plugin is a fatal error.

  1. uncomment the line in app/conf/configuration.php $_configuration['facebook_auth'] = 1;
  2. I had edit app/config/auth.conf.php for configuration

Whatever I am writing in the Facebook connexion image URL is given an error:

Fatal error: Uncaught Error: Class name must be a valid object or a string in /var/www/chamilo/www/main/admin/configure_plugin.php:88 Stack trace: #0 {main} thrown in /var/www/chamilo/www/main/admin/configure_plugin.php on line 88

What type of value is needed to the field Facebook connexion image URL?

ywarnier commented 4 years ago

Sorry for this. The Facebook plugin is outdated and doesn't use the correct Facebook SDK anymore. It is complicated for us to remove it from the Chamilo code between 2 major versions (one major version spans several years), so it is still there but it needs to be updated and nobody has really been asking for it, so it is veeeeeery low on our priorities list.

mesoshqip commented 4 years ago

Thank you Yannik Can you explain in which files you had integrated the sdk of facebook?

ywarnier commented 4 years ago

if you launch "composer update" (not on a system in production while running), you'll see a warning regarding the Facebook SDK if my memory serves.

The Facebook login code itself is located here: https://github.com/chamilo/chamilo-lms/tree/1.11.x/main/auth/external_login

mesoshqip commented 4 years ago

Thank you

ywarnier commented 4 years ago

Duplicate of #2273 and #2982

spacecabbie commented 4 years ago

Info: as of chamilo 1.11.12 no major errors are observed in basic install. But it will trow a error occasionally and more anoying it breaks login for everything else except facebook. Strongly recommend to at least provide a Warning about depreciation if a basic user enables it he might not be able to regain entry in the system. I will make a commit for that feel free to deny it if disagreed.