ElkArte Community

Elk Development => Feature Discussion => Exterminated Features => Topic started by: emanuele on December 29, 2013, 09:26:12 am

Title: Split Attachments.subs.php?
Post by: emanuele on December 29, 2013, 09:26:12 am
The file looks pretty large (3300 lines), so I thought it would be nice to have it smaller and "to the point":
https://github.com/emanuele45/Dialogo/commit/eef30a66101354e31ae4d9fd83dbf93d252f9e13

What do you think?
Title: Re: Split Attachments.subs.php?
Post by: Spuds on December 29, 2013, 01:04:11 pm
I'd say yea

Separating out the management actions from the rest makes sense to me
Title: Re: Split Attachments.subs.php?
Post by: TE on December 29, 2013, 01:09:28 pm
Quote from: Spuds – I'd say yea

Separating out the management actions from the rest makes sense to me
+1