Merge branch 'dev'

CHANGELOG.md

@@ -2,14 +2,22 @@
 
 This project uses [Semantic Versioning](http://semver.org/).
 
+## [1.8.0] -- 2016-09-20
+### Fixed
+- Added whitespace trimming to configuration parser.
+
+### Changed
+- Compatibility messages are now raised as debugging output.
+
 ## [1.7.3] -- 2016-08-03
 ### Fixed
 - Corrected global name parsing to account for UTF-8.
 - Updated API documentation.
 
 ## [1.7.2] -- 2016-07-20
 ### Fixed
-- Workaround for delayed bridge channel join caused by missing CHAT_MSG_CHANNEL_NOTICE events.
+- Workaround for delayed bridge channel join caused by missing 
+  CHAT_MSG_CHANNEL_NOTICE events.
 
 ## [1.7.1] -- 2016-07-20
 ### Changed
@@ -32,7 +40,8 @@ This project uses [Semantic Versioning](http://semver.org/).
 
 ## [1.6.6] -- 2015-12-16
 ### Fixed
-- A check is now done for officer status before sending a gratuitous officer announcement.
+- A check is now done for officer status before sending a gratuitous officer
+  announcement.
 - Added officer note data validation before parsing.
 
 ### Added
@@ -48,7 +57,8 @@ This project uses [Semantic Versioning](http://semver.org/).
 ### Changed
 - Refactored the user option validation.
 - Modified the GwHoldDown object.  Renamed `GwHoldDown:set()` to
-`GwHoldDown:start()` and created a new 'GwHoldDown:set()' mutator to change the interval.
+ `GwHoldDown:start()` and created a new 'GwHoldDown:set()' mutator to
+  change the interval.
 - Moved the Semantic Versioning parser to a LibStub library.
 
 ### Added
@@ -80,13 +90,15 @@ for handling hold-down expiry.
 
 ### Added
 - Added GwConfig object to contain configuration.
-- Added GwChannel objects for channel management, implementing transport and adaptation layers for communication.
+- Added GwChannel objects for channel management, implementing transport and 
+  adaptation layers for communication.
 - Added GwHoldDown and GwHoldDownCache objects.
 - Added LibStub libraries for SHA256, Salsa20, CRC16-CCITT, and Base64.
 
 ## [1.5.4] -- 2014-12-12
 ### Fixed
-- Corrected comparisons of character names to account for capitalization normalization in the API.
+- Corrected comparisons of character names to account for capitalization
+  normalization in the API.
 
 ### Changed
 - Updated luadoc.
@@ -191,8 +203,11 @@ flapping roster announcements for characters in peer co-guilds.
 
 ## [1.2.3] -- 2012-08-25
 ### Changed
-- Replaced the 32-bit string hash used to obfuscate channel names in the debugging output with a standard CRC-16-CCITT implementation to avoid overflow issues with `string.format()` in MoP.
-- Made some changes to the debugging code to improve visibility into message passing and replication.
+- Replaced the 32-bit string hash used to obfuscate channel names in the
+  debugging output with a standard CRC-16-CCITT implementation to avoid
+  overflow issues with `string.format()` in MoP.
+- Made some changes to the debugging code to improve visibility into message
+  passing and replication.
 
 ### Added
 - Added extra debugging information for current guild information.
@@ -230,7 +245,8 @@ flapping roster announcements for characters in peer co-guilds.
 ### Added
 - Added officer chat support.
 - Added message queuing.
-- Added a GwIsOfficer() check to the officer chat configuration phase to avoid pointless work.
+- Added a GwIsOfficer() check to the officer chat configuration phase to
+  avoid pointless work.
 - Added broadcast message type.
 - Added broadcasts of guild join and leave events.
 - Added broadcasts of promote and demote messages.
@@ -273,7 +289,8 @@ flapping roster announcements for characters in peer co-guilds.
 ## [1.1.03] -- 2011-01-14
 ### Changed
 - Moved `GuildRoster` call to PLAYER_LOGIN handler.
-- Limited conditions under which reinitialization occurred on PLAYER_GUILD_UPDATE.
+- Limited conditions under which reinitialization occurred on
+  PLAYER_GUILD_UPDATE.
 - Renamed `gwPlayerGuild` to `gwGuildName`.
 - Cleaned up prep/refresh/join flow for connecting to the common channel.
 
@@ -328,7 +345,8 @@ flapping roster announcements for characters in peer co-guilds.
 
 ## [1.0.15] -- 2010-12-03
 ### Fixed
-- Removed faulty `tContains()` for `gwPeerTable` checks to stop prolific kicking.
+- Removed faulty `tContains()` for `gwPeerTable` checks to stop
+  prolific kicking.
 
 ## [1.0.14] -- 2010-12-03
 ### Fixed
@@ -412,7 +430,8 @@ flapping roster announcements for characters in peer co-guilds.
 
 ## [1.0.02] -- 2010-11-12
 ### Added
-- Added container IDs to channel messages to avoid duplicates within the same co-guild.
+- Added container IDs to channel messages to avoid duplicates within the
+  same co-guild.
 
 ## [1.0.01] -- 2010-11-12
 ### Added
@@ -453,6 +472,7 @@ flapping roster announcements for characters in peer co-guilds.
 ## 0.9.00 -- 2010-11-01
 Initial commit.
 
+[1.8.0]: https://github.com/AIE-Guild/GreenWall/compare/v1.7.3...v1.8.0
 [1.7.3]: https://github.com/AIE-Guild/GreenWall/compare/v1.7.2...v1.7.3
 [1.7.2]: https://github.com/AIE-Guild/GreenWall/compare/v1.7.1...v1.7.2
 [1.7.1]: https://github.com/AIE-Guild/GreenWall/compare/v1.7.0...v1.7.1

Compat.lua

@@ -44,12 +44,12 @@ function gw.EnableCompatibility()
         if ElvUI[3].chat.enable then
             local ElvUIChat = ElvUI[1]:GetModule('Chat')
             gw.ChatFrame_MessageEventHandler = ElvUIChat.ChatFrame_OnEvent
-            gw.Write('ElvUI compatibility enabled.')
+            gw.Debug(GW_LOG_NOTICE, 'ElvUI compatibility enabled.')
         end
     elseif IsAddOnLoaded('Prat-3.0') then
         -- Use Prat's event handler for sending messages to the chat windows
         gw.ChatFrame_MessageEventHandler = function (...) Prat.Addon.ChatFrame_MessageEventHandler(Prat.Addon, ...) end
-        gw.Write('Prat-3.0 compatibility enabled.')
+        gw.Debug(GW_LOG_NOTICE, 'Prat-3.0 compatibility enabled.')
     end
 end
 

Config.lua

@@ -140,6 +140,10 @@ function GwConfig:load()
         end
         return estr
     end
+
+    local function trim(s)
+        return string.gsub(s, '^%s*(.-)%s*$', '%1')
+    end
 
     local function get_gm_officer_note()
         if not gw.IsOfficer() then
@@ -214,8 +218,12 @@ function GwConfig:load()
             if buffer ~= nil then
 
                 self.cversion = 1
-                buffer = strtrim(buffer)
-                local field = { strsplit(':', buffer) }
+
+                -- Groom configuration entries.
+                local field = {}
+                for i, v in ipairs({ strsplit(':', buffer) }) do
+                    field[i] = trim(v)
+                end
 
                 if field[1] == 'c' then
                     -- Guild channel configuration

GreenWall.toc

@@ -2,13 +2,13 @@
 ## Title: GreenWall
 ## Notes: Common communication channel as a replacement for guild chat in guild confederations.
 ## Author: Mark Rogaski <[email protected]>
-## Version: 1.7.3
+## Version: 1.8.0
 ## URL: https://github.com/AIE-Guild/GreenWall
 ## URL: http://wow.curse.com/downloads/wow-addons/details/greenwall.aspx
 ## DefaultState: enabled
 ## SavedVariablesPerCharacter: GreenWall,GreenWallLog
 ## X-Category: Guild
-## X-Date: 2016-08-03
+## X-Date: 2016-09-20
 
 Lib\Load.xml
 Constants.lua