Search results

The underscore is/was there because once you hit day 2, a casual ascension was just aftercore as far as any of my scripts should have been concerned. I assume I did it by looking at the link at ascension time, Valhalla manager? I believe can_interact was not usable because it doesn't work on...

It requires permissions. I have code somewhere that uses it when possible but I don't have those permissions in every clan I've been in.

Yeah, automatic opening is totes a bad idea. I can't imagine there is any other good way to make a faxbot response. Chat messages were probably a good UX and who knows if automation had really been considered (I'm going to guess the faxbots were not made with mafia as a consideration). But, I...

I think there has been an issued raised that mafia can not detect if chat is open. Although, I suppose you could track requests to newchatmessages.php (not sure if that is it) but I haven't really thought it all out either. But yeah, it seems like you have the right idea and checking the...

This is a tricky one. I think the existing workarounds don't handle having to change a faxbot well at all (in fact, having private clans is yet another workaround for this to avoid relying on a missing faxbot). The faxbot() function is very unique compared to other functions because it has an...

Make sense, real life stuff is always a priority. I'm guilty of similar, it does provide that push to get stuff done. Thanks, I appreciate the response! I completely understand the need for a code review and will get my stuff sorted out for you to look at.

Maybe that is why I feel like the scripting community is quite large as I'm privy to hearing from unshared maintenance. I never did bother to check though. The way players bring up issues (referring to the stone wool thing) is not always conducive to getting those things fixed. I didn't know...

I advise wrapping have_skill and always doing an is_unrestricted check.

My point is that you should at least try to ask first. Orphans get inherited by init and their return values are ignored.

At least one of those, CHIT, is actively updated (it updated today actually). At least ask first. And if you are going to fork them, try not to make them incompatible with the original.

Very true. My general stance is, as long as the original author hasn't explicitly said no, that with a sufficient good-faith attempt to contact (of course, what qualifies as sufficient and good-faith) is probably reasonable. There are further issues involving if a fork is a dramatic change...

If the spading of ensorcel ever finishes, I have code for a mafia patch that handles ensorcel (as a preference).

Fair enough. When I took over snapshot, it was after being prodded about it by other players for over a year. I made a few attempts to contact bumcheekcity about it but never heard back from him, I think I gave him 2 months (it was more of me sending him a kmail or seeing if a clannie knew how...

This was posted on April 1st and I can't tell if it is an Apple [sic, I'm not fixing it] Fools joke or not.

I use an afterAdventure script to handle these in my automation. But that may be more heavy-handed than you want.

I don't think it is terribly important anyway, unless we all lose our arms in a new path. I know that I started with making error messages more verbose. Some messages, such as "You can't make that item", are not as helpful in the middle of a script (but are just fine if you were using the...

Ah, neat. I can work with that. I was worried my recent modifier tinkering broke something. I see that the way the modifiers are handled, the unarmed property would end up applying to the set of modifiers from _voteModifiers. In my own build, I changed _voteModifiers to refer to the specific...

I just recently had an issue where mafia did not recognize "+20 Damage to Unarmed Attacks" as my _voteLocal1 modifier. It looks like it was recognized properly by VOTE_PATTERN but may have failed the call to Modifiers.parseModifier() - I can't really test it any more since my votes were already...

r19122 has Blood Blade as skill 5041 instead of 5042 (from r19121 update).

I believe this is sufficient for tracking the banisher, or at least it has been working with me (unless I missed some of the code in my svn diff check): Index: src/net/sourceforge/kolmafia/request/FightRequest.java =================================================================== ---...