Certain MM patch not applying

[ProgrammerFailure]

This mm patch does not seem to work:

Details

@part:HAS[@module[ModuleB9PartSwitch],#&/ModuleB9PartSwitch/baseVolume&>0]:FINAL
{

%MODULE

{

    name = WBIConvertibleStorage

    enableLogging = True

    showGUI = True

    confirmResourceSwitch = True

    defaultTemplate = LFO

    templateNodes = STORAGE_TEMPLATE

    templateTypes = Storage

    fieldReconfigurable = True

    resourcesToReplace = ALL

    capacityFactor = (#&/ModuleB9PartSwitch/baseVolume&)/1801

    logoPanelTransforms = decalPanel001, decalPanel002, decalPanel003, decalPanel004

    decalsVisible = false

    baseStorage = .001

    maxStorage = 9100

}

@MODULE[ModuleB9PartSwitch]
{
    %baseVolume = 0
}

}

All other MM patches seem to function
Here is the MM log:
MMPatch.log

[Lisias]

[scratch that]

[Lisias]

Ok. Let's try again.

@part:HAS[@module[ModuleB9PartSwitch],#&/ModuleB9PartSwitch/baseVolume&>0]:FINAL
{
	%MODULE
	{
		name = WBIConvertibleStorage
		enableLogging = True
		showGUI = True
		confirmResourceSwitch = True
		defaultTemplate = LFO
		templateNodes = STORAGE_TEMPLATE
		templateTypes = Storage
		fieldReconfigurable = True
		resourcesToReplace = ALL
		capacityFactor = (#&/ModuleB9PartSwitch/baseVolume&)/1801
		logoPanelTransforms = decalPanel001, decalPanel002, decalPanel003, decalPanel004
		decalsVisible = false
		baseStorage = .001
		maxStorage = 9100
	}
	@MODULE[ModuleB9PartSwitch]
	{
		%baseVolume = 0
	}
}

You want to add WBIConvertibleStorage to every PART that have B9PS and those baseVolume attribute is bigger than 0.

Well, I can't tell if the thing is working as intended only with the MM log. I need the ConfigCache too, so I can inspect whatever is there to see why this patch didn't worked as intended.

Obviously, this patch will not be applied on parts without the :HAS satisfied - are you absolutely sure the patch applying B9PS was applied first? You see, :FINAL loses it's meaning if everybody uses it: patches are applied in alfabetical order when having the same priority.

For example, the following patches WILL NOT work are intended:

<file:GameData/AnAddOnStartingWIthA>
@PART[*]:HAS[@MODULE[ModuleB9PartSwitch]]:FINAL
{
	yada yada yada
}

<file:GameData/BAddonStartingWithB>
@PART[*]:HAS[!@MODULE[ModuleB9PartSwitch]]:FINAL
{
	%MODULE[ModuleB9PartSwitch]
	{
		yada yada yada
	}
}

Because the first patch will be ran first for sure, when the B9PS patch below wasn't executed yet.

[ProgrammerFailure]

Forgot to mention, the patch is in the My_Patches file, but I have also tried it in a separate file, and when I did that, the file did not show up in "MMPatch.log". When I checked the ConfigSHA file, it was recorded by MM.

Here is ConfigCache and ConfigSHA
ConfigCache.txt
ConfigSHA.txt

[ProgrammerFailure]

Regarding order of patches, I have the near future mods which use B9PS. Those have part definitions, so it should apply before the :FINAL right? Those parts have functional B9PS switchers.

[Lisias]

Forgot to mention, the patch is in the My_Patches file, but I have also tried it in a separate file, and when I did that, the file did not show up in "MMPatch.log". When I checked the ConfigSHA file, it was recorded by MM.

Attach the whole patch file too. I think things are slightly simpler than it looks.

If I would write that pach, I would write it as:

@PART[*]:HAS[@MODULE[ModuleB9PartSwitch],#&/ModuleB9PartSwitch/baseVolume&>0]:FINAL
{
	%MODULE
	{
		name = WBIConvertibleStorage
		enableLogging = True
		showGUI = True
		confirmResourceSwitch = True
		defaultTemplate = LFO
		templateNodes = STORAGE_TEMPLATE
		templateTypes = Storage
		fieldReconfigurable = True
		resourcesToReplace = ALL
		capacityFactor = (#&/ModuleB9PartSwitch/baseVolume&)/1801
		logoPanelTransforms = decalPanel001, decalPanel002, decalPanel003, decalPanel004
		decalsVisible = false
		baseStorage = .001
		maxStorage = 9100
	}
	@MODULE[ModuleB9PartSwitch]
	{
		%baseVolume = 0
	}
}

Case matters.

[Lisias]

As well the location of the file. Tell me the full patchname of the patchfile too.

[ProgrammerFailure]

Full patch:
My_Patches.txt
The location of the file is "D:\Games\KSP instances\Kerbal Space Program\GameData\My patches\My_Patches.cfg"

[Lisias]

Well, the best way to see what happens is running it on my rig. If course no patches will be applied, as I don't have anything installed on it, but I will be able to at least see them being deleted, what I didn't found on your log.

I will be back to you in half an hour give or take.

[ProgrammerFailure]

I will try your updated patch soon

[Lisias]

Weird... I installed B9PS on my rig just to see the patching happening and... you appears to be right. MM is silently ignoring the patches mentioning baseVolume, but it's applying the others.

I think there's a small syntax error somewhere on this file. MM is absolutely tragic on handling errors. It's really shameful.

[Lisias]

Uhh... WAIT... Why are you using &? If you want the value of something you use #$....$ (dollar sign)!

& is used for boolean operations on conditional clausues (:HAS, etc) and for creating nodes...

[Lisias]

The patch looks right (once you use $ instead of &). I had toyed a bit with this on Forum once.

However, it is being ignored on my rig too.

[ProgrammerFailure]

I wrote the patch after checking the Sarbian MM wiki, where it said to use &. Must have misinterpreted something. So I should replace & with $? I'll test that when I get home.