Ticket #3260 (new task)

Opened 17 months ago

Last modified 6 months ago

Remove unused code

Reported by: Liorithiel Owned by: asterix
Priority: normal Milestone: 0.13
Component: xmpppy Version:
Severity: normal Keywords: refactoring
Cc: OS: All

Description

xmpp/ directory is full of unused code. Much of it belongs to the old xmpppy lib, which Gajim forked. There's a lot of pieces that were probably never tested by their authors (like adhoc commands -- last year i have found several misspellings, which means that code was not even compiled). To keep code clean, this code should be removed.

Attachments

Change History

Changed 17 months ago by roidelapluie

I had problems with privacy lists too and I don't think the related code has been tested.

Changed 17 months ago by Liorithiel

  • keywords refactoring added

Changed 17 months ago by steve-e

Let us go ahead. Can you please specify the parts of code which should be deleted.

Changed 17 months ago by Liorithiel

That's to be checked. Disco and adhoc for sure. One could integrate _nb files into the original ones, that would probably remove some code too. It looks like gajim doesn't use roster plugin from xmpp.

Changed 15 months ago by nk

liori did you rm the unused code so ticket gets closed?

Changed 15 months ago by anonymous

  • milestone set to 0.12

Changed 15 months ago by asterix

  • milestone 0.12 deleted

Changed 11 months ago by steve-e

What about Proxy65Manager and its related classes? Are they finished, have they been tested?

There are at least two errors:

Changed 11 months ago by steve-e

  • milestone set to 0.13

Changed 6 months ago by tomk

(In [9776]) initial cleanup of xmpppy perfomed, see #3260

Changed 6 months ago by tomk

I did some cleanup in branches/bosh_support@9776. All was done in xmpp/ folder except for two changes of variable paths in common/connection.py. To summarize it:

  • filetransfer.py, session.py, browser.py, commands.py deleted
  • methods from CommonClient? moved straight to NBCommonClient and deleted everything from client.py except for PlugIn? class and debug colors.
  • two constants from transports moved to transports_nb, transports.py deleted
  • methods of Bind and ComponentBind? classes from auth moved to NBBind resp. NBComponentBind in auth_nb, deleted auth.py
  • moved stuff from roster to roster_nb and roster.py deleted
  • moved four constants from features to features_nb and features.py deleted
  • update of imports, init.py

TODO: rename client.py to plugin.py (and change all related import statements)

I merged current (9775) trunk to bosh_support branch just before I did the cleanup. Session tests succeeded and Gajim was working well, but I tried just a few things.

Add/Change #3260 (Remove unused code)

Author



Change Properties
<Author field>
Action
as new
as The resolution will be set. Next status will be 'closed'
to The owner will change. Next status will be 'new'
The owner will change to anonymous. Next status will be 'assigned'
 
Note: See TracTickets for help on using tickets.