Bug in Beta 3305: PropertiesProxy object

Scripting
1

In the latest version of the beta (3.3 build 3305) line 3 of this code now produces an error. I’ve been using this for years. It seems like the issue is with GlyphsApp.PropertiesProxy (the type of .properties), because the issue doesn’t seem to occur when I set .properties to be a Python list of the same font info value objects.

newInstance = GSInstance()
newInstance.properties = Glyphs.font.instances[0].properties

Screenshot 2024-05-05 at 10.29.43 AM
Screenshot 2024-05-05 at 10.29.43 AM1338×204 78.2 KB

2

What exactly are you trying to do?

3

Just create a temporary instance that shares all or most of the info as an existing instance. So I want to copy all of the properties from the existing instance to the new one. I also do this when creating an instance from a master.

4

I’ll have a look.

5

Fixed it.

6

Thanks, Georg!

7

I’m he update is out.

8

Georg, the new beta update has a different bug with the same code! If you run the same 2-lines of code I provided it now produces this error:

image
image1324×208 58.7 KB

I use this code in Font Proofer’s Glyphs plugin. Every time you release a beta with a regression like this I need to stop what I’m doing and spend hours finding a workaround and releasing a patch update ASAP. So many Glyphs users use the public beta, and they expect it to be stable (even if that’s unrealistic). Could you send me a version to test before releasing it publicly? And going forward would you consider having a nightly/alpha program for plugin developers?

9

I’ve been experiencing a possibly related issue. I run a quick script just to fill in some font info fields and I have been getting the following error:

Screen Shot 2024-05-09 at 3.04.34 PM
Screen Shot 2024-05-09 at 3.04.34 PM1102×142 47.1 KB

1 Like
10

@nowell
Thanks. Fixed it.

@Graeme
can you show your code?

1 Like
11

Sure:

# MenuTitle: Cambridge Production
MacroTab.title = ""

def addPropertyToFont(font, key, value):
    while font.propertyForName_(key):
        font.removeObjectFromProperties_(font.propertyForName_(key))
    prop = GSFontInfoValueSingle()
    prop.key = key
    prop.value = value
    font.properties.append(prop)

for font in Glyphs.fonts:
    # speed up processing the font by disabling UI updates:
    font.disableUpdateInterface()
    
    if not "Cambridge" in font.familyName:
        continue
        # skip if it is not the font we want
    
    # PROPERTIES
    # convenience, see API docs: https://docu.glyphsapp.com/#gsfont
    font.copyright = "."
    font.manufacturer = "."
    
    key = "versionString"
    value = "Version %d.%03d"
    addPropertyToFont(font, key, value)    
    
    # FONT INFO
    parameters = {
        "Use Extension Kerning": True,
        "Export Mac Name Table Entries": True,
        "Use Typo Metrics": True,
        "Get Hints From Master": font.masters[0].id,
        "typoAscender": 1020,
        "typoDescender": -340,
        "typoLineGap": 0,
        "hheaAscender": 1020,
        "hheaDescender": -340,
        "hheaLineGap": 0,
        "winAscent": 1400,
        "winDescent": 340,
        }
    for parameterName in parameters.keys():
        value = parameters[parameterName]
        font.customParameters[parameterName] = value

        
    # GLYPH OR    
    # reenable UI updates:
    font.enableUpdateInterface()
12

Thanks for the script. I’ll add the missing method.
But you can simplify the script now:
instead of

    prop = GSFontInfoValueSingle()
    prop.key = key
    prop.value = value
    font.properties.append(prop)

you can do:

    font.properties[key] = value
13

Thank you, Georg, that makes a lot of sense!