Using () and [] as tags for alts and mains

by **reshurc** » Wed Feb 14, 2007 10:35 am

Having a slight problem.
We're useing Roster 1.7.2.

We have the following comment standard:
SWE Eng/Enc (Reshurc) <-- alts, Nationality, Proffessions, (is alt)
SWE A/H [Main] <-- for mains.

I have the following settings in Mains/Alts
Regex :: /([\(\w]+\))/i
Apply on field :: Public Note
Use match no :: 1
Main identifier :: [Main]
No result :: Main
Invalid result :: Mainless Alt
Alt of Alt ::Try to resolve

Anything I'm doing wrong here?

URL: http://www.lawguild.eu/lawroster/index.php

by **zanix** » Wed Feb 14, 2007 11:04 am

()[], these symbols have special meaning in reg_ex
Make sure you escape them

Code: Select all: \( \) \[ \]

The original /([\w]+)/i is a match on any word
The way you have it, you are trying to match the entire string (CharName) with a character name which will never work because of the extra ()
Since you are separating the name with defined symbols, added those symbols to the outside of /([\w]+)/i

Code: Select all: /\(([\w]+)\)/i

by **reshurc** » Wed Feb 14, 2007 1:23 pm

That worked =)
Still get these though

Warning: Division by zero in lawroster/addons/AltMonitor/altlist.php on line 777

Warning: Division by zero in lawroster/addons/AltMonitor/altlist.php on line 777

by **PleegWat** » Wed Feb 14, 2007 2:39 pm

That is a known bug, that originates in the real memberslist.php. I've since taken over that fix in AltMonitor, but I haven't pushed a release since. I'll push one as sure as I've ascertained myself that roster 1.7.3 has been released.

EDIT: I've just confirmed zanix hasn't managed to push the 1.7.3 release yet. It's ready but I understand there are some technical problems. The next AM version will release as soon as roster 1.7.3 is out since it will require 1.7.3