e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
318 stars 213 forks source link

Remove and optimize (hardcoded) LANS #6

Open CaMer0n opened 11 years ago

CaMer0n commented 11 years ago

Remove all hard-coded terms from e107 and optimize!

Translator's Checklist:

MikeyGMT commented 7 years ago

Looking at lans for updateadmin.php mostly duplicates. In fact why not send admin to usersettings? Seems like redundant code?

tgtje commented 7 years ago

Question how to proceed with these : see issue #2659
Explanation : on https://github.com/e107inc/e107/pull/2232/commits/2c68d9ee52eaf52bb80478420560bd5ef9269df6#diff-d7f2372327954fe64546996f2f909c95 (closed/merged pull request) on oldpolls php some LAN's were replace by new LAN's (LAN_FROM & LAN_TO new > were pollan_38; and 37 (being commented out). Some other new ones were added and put in admin/lan_admin php... Now LAN_FROM and LAN_TO (representing 'from' and 'to' were never incorporated.. Should these LAN"s (pollan 37;38) be reinstated to use (and oldpoll php be recoded OR the new LAN"s be added to the poll/English php (LAN_FROM & TO )and the coding left untouched ? Please some guidance here or advice how to....

Moc commented 4 years ago

@CaMer0n was this one closed intentionally?

CaMer0n commented 4 years ago

@Moc No. Looks like it came with the pull-request.