This repository has been archived by the owner on Aug 28, 2020. It is now read-only.
src: GatewayDriver#register changes #480
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* refactor: No more _update
* cleanup: More re-organization
* src: More refactors
* Settings#{list,resolveString} -> Settings#display
* style: Rename constant name
* refactor: Re-add Conf#check and reduce duplicated code in Settings#reset
* fix: _resolveUpdateOverloads not parsing reset args correctly
* fix: Synchronize the Settings entry on update or reset instead of returning wrong results
* typings: Fixed QB types and updated everything
* docs: Added missing jsdocs and refactored Settings#get
* fix: conf and userconf always throwing on empty values
* fix: Better Schema#get
* fix: Settings#show using Map[key] instead of Map#get(key)
* fix: Settings#_resolveUpdateOverloads not checking for the lack of a key
* fix: conf command not passing all args for check
* fix: Fixed multiple things
* docs: Update guides to reflect latest changes
* pr: Update changelog
Now it's the time when I run and hide for such a silly mistake.
* wip: Settings rewrite
* design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard
* refactor: Made SettingsFolder#get use Array#reduce
* src: Add Settings#init
* src: Added SettingsFolder#display
* src: Remove Settings and all traces of it
* src: Update Gateway, move SettingsFolder#existenceStatus to Settings
* docs: Documented all methods and properties of Settings and SettingsFolder
* naming: Gateway#type -> Gateway#name
* src: Change all occurrences of Settings[key] to Settings.get(key)
* src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable}
* src: guildSettings[key] -> guildSettings.get(key)
* typings: GatewayStorage#type -> GatewayStorage#name
* refactor: Faster Schema#get
* typings: Fix Settings class typings
* typings: Iterable -> IterableIterator
* docs: Fixed util docs
* fix: Use Map#call instead of inherited SettingsFolder#get and Schema
* fix: Settings#init deepCloning the wrong value
* fix: KlasaGuild#language using Settings[key] instead of Settings.get(key)
* inspection: Make Settings#{id|base} non-enumerable
* fix: Sandbox Schema#get to mimic previous functionality
* fix: Better errors for Settings#{reset,update}
* fix: Fixed SettingsFolder#display
* docs: Added comments to justify the usage of Map.prototype.get.call
* fix: SettingsFolder#display not getting the right path
* fix: Revert change to commandHandler's mention only check
* docs: Justify the prefix.length in commandHandler's mentionOnly
* feat: SettingsFolder#pluck
* fix: Updated providers helpers for the new Gateway results
* fix: SettingsFolder#update not overloading well
* fix: Settings and SettingsFolder trying to access client
* fix: Fixed updating not patching correctly
* fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance
* src: Better relative path parsing, fixed update inserting full values into db
* fix: JSON provider not parsing create queries
* fix: Configuration commands not checking correctly
* src: Added Settings#target, Gateway#{create,acquire}, removed create in get
* fix: Init schedule before pieces (but after SG)
* i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors
* refactor: Better guild resolving for Settings#{reset,update}
* fix: resolveGuild calls without client
* fix: SQLProvider accessing to the gateway wrongly
* changelog: Updated for the PR
* fix: Settings#clone not working as expected
* misc: Make Settings#{id,gateway,target} enumerable
* revert: Settings#{gateway,target} enumerability
* cleanup: Remove GatewayStorage#defaults
* misc: Make Schema#defaults a property
* fix: Call this.init() in destroy instead of patching
* docs: Updated changelog
* lint: Fixed lint
* docs: Fixed jsdocs
* src: Reorganize files
* style: Remove optional property in Settings#id
* memory: Deepclone patch, not init
* fix: Access to undefined
* fix: Settings#update not serializing arrays correctly
* src: Remove all usage of deepClone in SettingsGateway
* misc: Make Schema#defaults a SettingsFolder instance
* docs: Finished changelog
* misc: Remove guild option from the conf command
* docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG
* perf: Optimize Gateway#sync to the clouds for empty schemas
* feat: Added `KlasaClient#settingsSyncEntry` event.
* docs: Correct PR number in CHANGELOG
* src: Emit settingsSyncEntry on broadcastEval
* fix: Entry settings not setting their existence status to true in shards
* misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`.
* misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate
* fix: coreSettingsUpdate is not an once event
* fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events
kyranet
added
Type: Enhancement
UnseenFaith
reviewed
Nov 10, 2018
UnseenFaith
reviewed
Nov 10, 2018
UnseenFaith
reviewed
Nov 10, 2018
UnseenFaith
force-pushed
the
settings
branch
from
5fbb249
to
5f3aef6
Compare
kyranet
added a commit
that referenced
this pull request
Nov 11, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
UnseenFaith
pushed a commit
to UnseenFaith/klasa
that referenced
this pull request
Nov 18, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (dirigeants#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (dirigeants#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (dirigeants#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (dirigeants#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (dirigeants#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Nov 20, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Nov 20, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Nov 21, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Nov 21, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Nov 21, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Dec 4, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
UnseenFaith
pushed a commit
to UnseenFaith/klasa
that referenced
this pull request
Dec 11, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (dirigeants#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (dirigeants#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (dirigeants#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (dirigeants#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (dirigeants#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
UnseenFaith
pushed a commit
to UnseenFaith/klasa
that referenced
this pull request
Dec 11, 2018
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (dirigeants#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (dirigeants#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (dirigeants#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (dirigeants#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (dirigeants#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Feb 11, 2019
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Feb 23, 2019
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Feb 24, 2019
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Mar 8, 2019
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
kyranet
added a commit
that referenced
this pull request
Apr 17, 2019
* refactor: Convert GatewayDriver into a Collection<string, Gateway> (#467) * refactor: Convert GatewayDriver into a Collection<string, Gateway> * fix: Better error message for gateway existence in register Co-Authored-By: kyranet <kyradiscord@gmail.com> * refactor: Upgrade API for Settings#update (#426) * refactor: No more _update * cleanup: More re-organization * src: More refactors * Settings#{list,resolveString} -> Settings#display * style: Rename constant name * refactor: Re-add Conf#check and reduce duplicated code in Settings#reset * fix: _resolveUpdateOverloads not parsing reset args correctly * fix: Synchronize the Settings entry on update or reset instead of returning wrong results * typings: Fixed QB types and updated everything * docs: Added missing jsdocs and refactored Settings#get * fix: conf and userconf always throwing on empty values * fix: Better Schema#get * fix: Settings#show using Map[key] instead of Map#get(key) * fix: Settings#_resolveUpdateOverloads not checking for the lack of a key * fix: conf command not passing all args for check * fix: Fixed multiple things * docs: Update guides to reflect latest changes * pr: Update changelog * fix: Collection not being destructured from discord.js Now it's the time when I run and hide for such a silly mistake. * Settings rewrite (#471) * wip: Settings rewrite * design: Chain SettingsFolder#get instead of recursing, remove '*' wildcard * refactor: Made SettingsFolder#get use Array#reduce * src: Add Settings#init * src: Added SettingsFolder#display * src: Remove Settings and all traces of it * src: Update Gateway, move SettingsFolder#existenceStatus to Settings * docs: Documented all methods and properties of Settings and SettingsFolder * naming: Gateway#type -> Gateway#name * src: Change all occurrences of Settings[key] to Settings.get(key) * src: SettingsUpdateOptions#{action|avoidUnconfigurable} -> {arrayAction|onlyConfigurable} * src: guildSettings[key] -> guildSettings.get(key) * typings: GatewayStorage#type -> GatewayStorage#name * refactor: Faster Schema#get * typings: Fix Settings class typings * typings: Iterable -> IterableIterator * docs: Fixed util docs * fix: Use Map#call instead of inherited SettingsFolder#get and Schema * fix: Settings#init deepCloning the wrong value * fix: KlasaGuild#language using Settings[key] instead of Settings.get(key) * inspection: Make Settings#{id|base} non-enumerable * fix: Sandbox Schema#get to mimic previous functionality * fix: Better errors for Settings#{reset,update} * fix: Fixed SettingsFolder#display * docs: Added comments to justify the usage of Map.prototype.get.call * fix: SettingsFolder#display not getting the right path * fix: Revert change to commandHandler's mention only check * docs: Justify the prefix.length in commandHandler's mentionOnly * feat: SettingsFolder#pluck * fix: Updated providers helpers for the new Gateway results * fix: SettingsFolder#update not overloading well * fix: Settings and SettingsFolder trying to access client * fix: Fixed updating not patching correctly * fix: settingsUpdateEntry event receiving a SettingsFolder instead of the Settings instance * src: Better relative path parsing, fixed update inserting full values into db * fix: JSON provider not parsing create queries * fix: Configuration commands not checking correctly * src: Added Settings#target, Gateway#{create,acquire}, removed create in get * fix: Init schedule before pieces (but after SG) * i18n,fix: Default guild option in Settings#{update,reset}, i18n all errors * refactor: Better guild resolving for Settings#{reset,update} * fix: resolveGuild calls without client * fix: SQLProvider accessing to the gateway wrongly * changelog: Updated for the PR * fix: Settings#clone not working as expected * misc: Make Settings#{id,gateway,target} enumerable * revert: Settings#{gateway,target} enumerability * cleanup: Remove GatewayStorage#defaults * misc: Make Schema#defaults a property * fix: Call this.init() in destroy instead of patching * docs: Updated changelog * lint: Fixed lint * docs: Fixed jsdocs * src: Reorganize files * style: Remove optional property in Settings#id * memory: Deepclone patch, not init * fix: Access to undefined * fix: Settings#update not serializing arrays correctly * src: Remove all usage of deepClone in SettingsGateway * misc: Make Schema#defaults a SettingsFolder instance * docs: Finished changelog * misc: Remove guild option from the conf command * docs: Forgot to mention arrayPosition->arrayIndex in CHANGELOG * perf: Optimize Gateway#sync to the clouds for empty schemas * feat: Added `KlasaClient#settingsSync` event. (#475) * feat: Added `KlasaClient#settingsSyncEntry` event. * docs: Correct PR number in CHANGELOG * src: Emit settingsSyncEntry on broadcastEval * fix: Entry settings not setting their existence status to true in shards * misc: Renamed `KlasaClient#{settingsUpdateEntry,settingsDeleteEntry,settingsCreateEntry}` to `KlasaClient#{settingsUpdate,settingsDelete,settingsCreate}`. * misc: Rename prefix settingsUpdate core event to... coreSettingsUpdate * fix: coreSettingsUpdate is not an once event * fix: Old typings, patch before emitting events * fix: Settings delete core event (#477) * fix: Added core event handler for settingsDelete * docs: Updated IncludedEvents * docs: Correct changelog PR number * fix: Don't check this.client.shard in both core settings event handlers * misc: Add IS support for both core settings events handlers * misc: Much better and faster shard check for both settings events * src: GatewayDriver#register changes * fix: I missed this * misc: Requested changes * src: Requested change * fix: Who did this * fix: settings is not defined in the evals
4 tasks
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the PR
For better registering of custom gateways, the current
GatewayDriver#registerwas useless and needed a workaround. This PR fixes it by asking for an instance of GatewayStorage or extended, as discussed with @UnseenFaith.Also fixes a bug introduced in #475.
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)
Changed:
GatewayDriver#registerto take aGatewayStorageinstance.Removed:
Gateway#store.Semver Classification