Page 5 of 5 FirstFirst ... 3 4 5
Results 41 to 47 of 47

Thread: Vhaeraun's Glorious PVP Bookkeeper

  1. #41
    Developer Veracity's Avatar
    Join Date
    Mar 2006
    Location
    The Unseelie Court
    Posts
    11,161

    Default

    There were three or four threads, all reporting the same bug. I responded on all of them that I'd fix it, but didn't bother reporting back on all of them that I HAD fixed it.
    Ph'nglui mglw'nafh Cthulhu
    R'lyeh wgah-nagl fhtagn.

  2. #42
    Senior Member
    Join Date
    Jun 2016
    Posts
    126

    Default

    The new ‡ indicator means it's failing to recognize the checking it twice mini (at least, I think that's the problem, it could of course be something completely unrelated).

  3. #43
    Senior Member
    Join Date
    Jun 2011
    Posts
    405

    Default

    It also looks like it's not seeing the Holiday Shopping mini. By which I mean that that mini is not showing in my results table at all.

  4. #44
    Senior Member
    Join Date
    Jun 2011
    Posts
    405

    Default

    Regarding the unrecognized Holiday Shopping mini, I'll be damned if I can figure out why this regex isn't matching this page text:
    Code:
    string informationBoothContests = "<td valign=\"top\" nowrap><b>(.*?)\\**</b></td><td valign=\"top\">(.*?)</td><td valign=\"top\" align=\"center\">(\\d+|\\?\\?)</td>";
    Code:
    <td valign="top" nowrap><b>Holiday Shopping*</b></td><td valign="top">You know what makes a great holiday gift?  Gargle Blasters! (Who has earned the most Pan-Dimensional Gargle Blasters?)</td><td valign="top" align="center" colspan="1">3</td>
    Regarding the recognized-but-not-tallied Checking It Twice mini, it was also my suspicion that the new double dagger (?) note icon is messing it up. I'm sure this could be solved by using &Dagger; (with a capital 'D') somewhere, to handle it. But I don't even entirely understand how the existing asterisk note icon is being handled, so I'm not sure where to make the change.

  5. #45
    Senior Member
    Join Date
    Apr 2009
    Posts
    1,755

    Default

    Regarding the unrecognized Holiday Shopping mini, I'll be damned if I can figure out why this regex isn't matching this page text:
    Originally Posted by PaladinWhite View Post
    regex has: align="center">
    page has: align="center" colspan="1">


    And the * is handled by: \\**
    (zero or more asterisks).

    So you probably want something like:
    Code:
     (\\*|&Dagger;)*
    (modulo correct quoting and grouping)
    Last edited by xKiv; 12-04-2017 at 03:55 PM.

  6. #46
    Senior Member
    Join Date
    Jun 2011
    Posts
    405

    Default

    Thanks a million, xKiv.

    I count myself among those sorry masses that are just capable enough with regex to get in trouble. Don't know how I missed that colspan attribute. I broadened the regex to match any attributes in that last <td> tag. Hopefully that broadening doesn't cause other issues.

    I'm still not entirely sure what's up with that double-backslash, double-asterisk to match the literal asterisk in mini names. Something-something-double-escaping? But even then, I don't get where the second asterisk comes in. My head hurts. At any rate, editing &Dagger; into that same regex solved the issue.

    All-in-all: a one-line change. Here's the updated version.
    Attached Files
    Last edited by PaladinWhite; 12-05-2017 at 07:24 PM. Reason: add attachment

  7. #47
    Senior Member
    Join Date
    Apr 2009
    Posts
    1,755

    Default

    I'm still not entirely sure what's up with that double-backslash, double-asterisk to match the literal asterisk in mini names. Something-something-double-escaping?
    Originally Posted by PaladinWhite View Post
    Yes. You need \\ in the source file to put a \ in the actual string value. And you need \* in the actual string value to match a literal asterisk.
    The second asterisk is just the normal greedy quantifier - match zero or more instances of the preceding "literal asterisk" (so it will match texts with no asterisk, with *, with **, with ***, etc...).

Similar Threads

  1. 13119: Add Glorious Blessing of the War Snapper.
    By RSS Bot in forum Latest SVN Changes
    Replies: 0
    Last Post: 11-09-2013, 06:21 PM
  2. Bug - Fixed Can't cast Song of the Glorious Lunch in mafia interface in 10758
    By Theraze in forum Bug Reports
    Replies: 4
    Last Post: 03-10-2012, 06:38 PM
  3. Bug - Fixed Steel Lasagna is not affected by Got Milk/Gourmand/Glorious Lunch
    By GValko in forum Bug Reports
    Replies: 1
    Last Post: 02-29-2012, 06:00 AM
  4. 10652: Check for Glorious Lunch before eating in Axecore
    By RSS Bot in forum Latest SVN Changes
    Replies: 0
    Last Post: 02-27-2012, 02:51 PM
  5. Replies: 1
    Last Post: 02-21-2012, 11:56 PM

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •