Thread Tools Display Modes
06-26-11, 04:23 PM   #1
Ketho
A Pyroguard Emberseer
 
Ketho's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2010
Posts: 1,026
Originally Posted by qod View Post
COMBAT_LOG_EVENT_UNFILTERED is not the only change that will affect addons when 4.2 goes live. I author an addon that used string.replace to remove unneeded spaces from a string, only to find that Blizzard decided to remove string.replace for some strange reason.
offtopic, but why didn't you just use strtrim / string.trim if you just wanted to remove "unneeded spaces"?
  Reply With Quote
06-26-11, 04:32 PM   #2
Nibelheim
local roygbi-
 
Nibelheim's Avatar
AddOn Author - Click to view addons
Join Date: Jan 2010
Posts: 1,600
Originally Posted by Ketho View Post
offtopic, but why didn't you just use strtrim / string.trim if you just wanted to remove "unneeded spaces"?
Trim only removes trailing and leading spaces. Replace was useful for converting "Buff Bar 1" to "BuffBar1".
  Reply With Quote
06-27-11, 12:24 AM   #3
Vlad
A Molten Giant
 
Vlad's Avatar
AddOn Author - Click to view addons
Join Date: Dec 2005
Posts: 793
Simply converting all string.replace/strreplace/:replace into "gsub" won't do as you got to escape certain characters. Yet it appears that Blizzard just did a mass "find and replace" on the few lines I read but maybe not, in any case addon authors gotta watch this and get familiar with how gsub and patterns work with lua for 4.2 I figure.

Strange isn't it, removing string.replace? I wonder why...
  Reply With Quote
06-27-11, 08:12 AM   #4
arith
A Cyclonian
 
arith's Avatar
AddOn Author - Click to view addons
Join Date: May 2009
Posts: 45
So is it confirmed the 4.2 will be on live server on June 28th?
  Reply With Quote
06-27-11, 10:37 AM   #5
Verttex
Super Monkey
 
Verttex's Avatar
AddOn Author - Click to view addons
Join Date: Jan 2010
Posts: 297
Originally Posted by arith View Post
So is it confirmed the 4.2 will be on live server on June 28th?
It has not been confirmed by Blizzard but we expect it to arrive then.
  Reply With Quote
06-27-11, 11:43 AM   #6
qod
A Murloc Raider
AddOn Author - Click to view addons
Join Date: May 2011
Posts: 6
Originally Posted by Ketho View Post
offtopic, but why didn't you just use strtrim / string.trim if you just wanted to remove "unneeded spaces"?
Originally Posted by Nibelheim View Post
Trim only removes trailing and leading spaces. Replace was useful for converting "Buff Bar 1" to "BuffBar1".
It was to only remove spaces in certain situations. Just the spaces before and after a dash, as in "Playername - Server Name" to "Playername-Server Name" because "Playername-ServerName" wouldn't work for me. I was using strreplace(stringname, " - ", "-") and now use string.gsub(stringname, "%s%-%s", "-") and get the same results.

I never came across the bug because my strings were never very long, as they were just a list of tanks that are currently in my party.
  Reply With Quote
06-27-11, 12:54 PM   #7
Xrystal
nUI Maintainer
 
Xrystal's Avatar
Premium Member
AddOn Author - Click to view addons
Join Date: Feb 2006
Posts: 5,961
Hmm, darn windows search isn't finding a single COMBAT_LOG_EVENT_UNFILTERED in the addons folder. That simply cannot be right. I knew there was a reason I stopped using that search system.

Edit:
Ah, programmers notepad to the rescue, haha. But thankfully the only ones of my own addons that use it are about 4 years old and I'm sure they're no longer used by now unless they have miraculously been working all these years with no problems.
__________________


Characters:
Gwynedda - 70 - Demon Warlock
Galaviel - 65 - Resto Druid
Gamaliel - 61 - Disc Priest
Gwynytha - 60 - Survival Hunter
Lienae - 60 - Resto Shaman
Plus several others below level 60

Info Panel IDs : http://www.wowinterface.com/forums/s...818#post136818

Last edited by Xrystal : 06-27-11 at 12:57 PM.
  Reply With Quote
06-27-11, 02:10 PM   #8
Seerah
Fishing Trainer
 
Seerah's Avatar
WoWInterface Super Mod
Featured
Join Date: Oct 2006
Posts: 10,860
There's a specific thing you need to do to make it search in Lua and XML files. Unfortunately, I forgot it as soon as I did it.
__________________
"You'd be surprised how many people violate this simple principle every day of their lives and try to fit square pegs into round holes, ignoring the clear reality that Things Are As They Are." -Benjamin Hoff, The Tao of Pooh

  Reply With Quote
06-27-11, 04:56 PM   #9
Vlad
A Molten Giant
 
Vlad's Avatar
AddOn Author - Click to view addons
Join Date: Dec 2005
Posts: 793
Notepad++ is free and has a "find in files" feature, I don't think windows search is really designed to look into files for strings, never used it for that myself in any case. :P

Btw very unhappy that major interface changes aren't officially announced in a compilation on the official forums, even the MVP aren't really doing it and I reckon if you are a Interface/addons MVP you'd have to focus and try report API changes and such between major patches. I don't know, maybe I demand too much.
  Reply With Quote
06-27-11, 04:58 PM   #10
Tekkub
A Molten Giant
 
Tekkub's Avatar
AddOn Author - Click to view addons
Join Date: Dec 2005
Posts: 960
You could just get on the PTR yourself and see if your addons broked. It's not that hard to do.
__________________
I have reached enlightment.
Thank you bacon!
  Reply With Quote

WoWInterface » Site Forums » News » Reminders before 4.2 goes live

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off