Suitecrm: Move Twitter OAuth to Composer, or remove it entirely

Created on 18 Jul 2019  路  6Comments  路  Source: salesagility/SuiteCRM

We seem to use this package: https://github.com/abraham/twitteroauth

It's in `include/social/twitter/twitter_auth/: https://github.com/salesagility/SuiteCRM/blob/0b7909a5e63810170cf81a744ef849d03b1291c4/include/social/twitter/twitter_auth/

Based on this line, the 0.2.0-beta2 version, though that doesn't seem right because 0.2.0-beta2 came out in December of _2009_: https://github.com/salesagility/SuiteCRM/blob/0b7909a5e63810170cf81a744ef849d03b1291c4/include/social/twitter/twitter_auth/twitteroauth/twitteroauth.php#L35

Surprisingly, Twitter is still on the same API version (1.1) so _I think_ it might still work?

If it doesn't actually work, we should probably just remove this functionality entirely since it doesn't seem anyone is using it. Ditto for the Facebook integration.

Important Bug

Most helpful comment

@pgorod just based on my understanding of the Twitter API's various breaking changes over the years and the fact that this package is so old, I'd be incredibly surprised if it actually works today. I'd be happy to see evidence suggesting otherwise, though.

All 6 comments

Based on https://github.com/salesagility/SuiteCRM/issues/2914 and https://github.com/salesagility/SuiteCRM/issues/2644, neither Twitter nor Facebook integration works. I'd say the best course of action is just to remove them unless someone wants to step up, fix them, and move the packages to Composer.

@isleshocky77 I know you've mentioned being interested in this functionality, do you want to fix it or should we remove it?

I would like the feature of working with twitter, but I have no idea what this does and I'm guessing if someone were to do it now they would do it completely differently. I would say just remove it all together and it could always be added back when done properly.

I think I had more mentioned it because I was trying to do something similar with an internal API.

I'm not optimistic about this idea of removing features - we have no way of knowing how many people are using it, and how important it is to them. Sometimes I see people mentioning these features on the Forums. Maybe it's not broken for everybody?

@pgorod just based on my understanding of the Twitter API's various breaking changes over the years and the fact that this package is so old, I'd be incredibly surprised if it actually works today. I'd be happy to see evidence suggesting otherwise, though.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

darouca picture darouca  路  3Comments

ArturoBurela picture ArturoBurela  路  3Comments

Mausino picture Mausino  路  3Comments

likhobory picture likhobory  路  3Comments

Vhex picture Vhex  路  3Comments