With reference to this post:
http://forums.cgsociety.org/showthread.php?f=6&t=980443
The Onion isn’t working in Max 2012 – it has script errors.
Paul Hormis (Hyperrent) has been looking after the Onion of late, but hasn’t updated it for 2012 and hasn’t replied to my pathetic whinging begging emails.
I’m wondering if it’s just a simple case of chnaging a couple of lines of code that isn’t supported by Max 2012 any more…
Is this something anyone who’s on the case can look into?
I will give birth to your children if you do it. And I’m a man.
edit:
Fixed by Paul Hormis who I believe is now the official custodian of The Onion.
I think this comes from the 2012 paramStr rolloutCreator bug.
Basically what you need to do is open \Scripts heOnion heOnion_resource\OnionStructure.ms
and add a space at the end of all paramStr’s.
examples:
replace
paramStr:“state” with paramStr:”state “
replace
paramStr:LayActString with paramStr:(LayActString+” “)
Haven’t tested it but could work…
Thx lo,
that worked for me
Didn’t work for me! Though I’m sure I’m doign soemthing wrong… many thanks for looking into it though!
I noticed there are other paramstr’s floating around:
paramStr:LayWireString
paramStr:“col” codestr:
etc.
Perhaps lo/z buffer could assist by posting his correctly edited code…?
much love!
yes, you need to change ALL instances of paramStr, not just the two examples I showed.
I’d feel bad posting the code, as it’s not my tool.
Go on – post the code! :bowdown:
I don’t think Paul Hormis has time to sort it out, and I’m not even sure if it’s his tool either – he just picked up the batton like a hero.
PM me if you feel squeamish about publicly posting?
I’m still getting script errors!
Hey guys, i was able to solve this
Just ditch all modifications in your TheOnion source files and fix “rolloutCreator.ms” itself.
You just have to add two lines after Line 163 and 192 in this script for it to work:
in \stdscripts\baselib\rolloutCreator.ms goto line 163
after line 163 , insert the following NEW line, which i colored orange here
Line 163: append str paramStr
Line 164: [B]append str space -- missing space bugfix[/B]
to the same after line 193 ( previously was Line 192, but we just inserted a line , capiche ?
again, the new line is colored in orange here
Line 193: append str (paramStr as string)
Line 194: [B]append str space -- missing space bugfix[/B]
Of course make a backup of “rolloutCreator.ms” in some other folder (or this script might be run twice)
I would also insert that “bugfix” comment too, this makes it easier later on to find it again…
I quickly tested TheOnion after that, and it seemed to be working
PS:
credits for this fix go to Duncan Howdin and James Haywood which reported the problem and initially posted the fix on some secret Max board 😈
@mynewcat : does spacefrog’s fix work for you? If not I’ll find the file I edited and PM it to you
Your fix did work, I restarted a second time after making the second load of changes – after giving up I accidentally hit my O (for Onion key) and it popped up working.
If I get time tomorow I will edit and save that file based on spacefrog’s edits and make it available for anyone who searches for the fix so they can patch their Max 2012.
Once again, many thanks for your help – for me, on certain projects using the onion is like having running shoes versus wading through mud in boots.