openstack
/
ptgbot
Code Issues Proposed changes

Replace ~wipe by more selective ~reload 

Wiping the database was no longer very useful. Replace this
with a ~reload action that reload base data from configuration.

Change-Id: Iaa55a2060053f77dd2ab1072e59b21a28812f213
This commit is contained in:
Thierry Carrez 2018-02-13 15:19:59 +01:00
parent f8ce01d4e5
commit 6b408c46de
3 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
README.rst
View File

@ -127,8 +127,8 @@ You have to be a channel operator (+o) to use admin commands.
~newday ~newday
Removes now/next/location entries, to be run at the start of a new day Removes now/next/location entries, to be run at the start of a new day
~wipe ~reload
Resets the database entirely (removes all defined tracks and topics) Resets the database entirely (reloads from configuration)
Local testing Local testing

4
ptgbot/bot.py
View File

@ -163,8 +163,8 @@ class PTGBot(irc.bot.SingleServerIRCBot):
return return
words = msg.split() words = msg.split()
command = words[0][1:].lower() command = words[0][1:].lower()
if command == 'wipe': if command == 'reload':
self.data.wipe() self.data.reload()
elif command == 'newday': elif command == 'newday':
self.data.new_day_cleanup() self.data.new_day_cleanup()
elif command == 'list': elif command == 'list':

3
ptgbot/db.py
View File

@ -167,8 +167,9 @@ class PTGDataBase():
self.data['location'] = {} self.data['location'] = {}
self.save() self.save()
def wipe(self): def reload(self):
self.data = copy.deepcopy(self.BASE) self.data = copy.deepcopy(self.BASE)
self.load_data_from_config()
self.save() self.save()
def save(self): def save(self):