Сбой записи XPS с.Net 4 / ошибка рендеринга шрифтов?

Question

У нас есть приложение WPF, которое позволяет пользователям вводить и стилизовать текст по своему желанию. Их текст затем преобразуется в XPS для вставки в PDF с использованием ABCPdf.

Недавно мы перешли на.Net 4, и теперь поколение XPS иногда генерирует плохо сформированный XPS.

Единственный раз, когда генерируется неправильно сформированный XPS, это когда пользователь использует шрифт, который перекрывает два символа (например, Sevillana, доступный в Google Fonts). В.Net 4 символы перекрываются, а в.Net 3.5 - нет. Из другого SO вопроса я понял, что это потому, что.Net 4.0 изменил свой механизм рендеринга шрифтов.

.Net 3.5:

.Net 4.0: (обратите внимание, что буквы "d" и "A" перекрываются)

Похоже, что.Net 4 не учитывает расстояние между апострофами или даже отрицает его. Если я уберу апостроф, интервал увеличится. Я создал простой тестовый проект для демонстрации проблемы: https://github.com/tbroust-trepia/wpf-4-font-rendering

Наш метод сохранения FlowDocument в XPS почти такой же, как при создании документа XPS из FlowDocument и присоединении его на лету (я подозреваю, что первоначальный разработчик просто скопировал / вставил этот код); он просто сохраняет XPS в файл, а не в поток.

Инструмент MS IsXPS говорит, что определенный узел недействителен. На самом деле, в Documents\1\Pages\1.fpage значение Индексов равно ;,-16;,84;;;;,30, Я вижу, что "0.-16" на самом деле не число.

Итак, у меня много вопросов:

• Почему это происходит? Я не понимаю, как значения преобразуются в эти индексы. Я полагаю, что механизм преобразования пытался установить его на "-0.16", но все испортилось?
• Могу ли я изменить / проверить значение этого индекса до его сохранения?
• Если я не могу, как я могу проверить потом? Я нашел здесь своего рода валидатор XPS, но я не понимаю, как я могу оттуда проверить значение индексов. Я мог бы сделать это вручную, читая и анализируя XML, но я действительно уверен, что это не очень хорошая идея.

редактировать

Я открыл ошибку в Microsoft, но я не ожидаю многого там. Я думаю, что я попытаюсь изменить сгенерированный XPS. Это грязно, но это может сработать.

Answer 1

В ожидании MS сделать все, что они хотят, я нашел способ исправить сломанный XPS.

Предупреждение: грязный взлом впереди

public class XpsFile
{
    /// <summary>
    /// Regex to validate XPS "indices" property (gotten from ABCPDF)
    /// </summary>
    private static readonly string IndicesRegex = @"(((\(([1-9][0-9]*)(:([1-9][0-9]*))?\))?([0-9]+))?(,(\+?(([0-9]+(\.[0-9]+)?)|(\.[0-9]+))((e|E)(\-|\+)?[0-9]+)?)?(,((\-|\+)?(([0-9]+(\.[0-9]+)?)|(\.[0-9]+))((e|E)(\-|\+)?[0-9]+)?)?(,((\-|\+)?(([0-9]+(\.[0-9]+)?)|(\.[0-9]+))((e|E)(\-|\+)?[0-9]+)?))?)?)?)(;((\(([1-9][0-9]*)(:([1-9][0-9]*))?\))?([0-9]+))?(,(\+?(([0-9]+(\.[0-9]+)?)|(\.[0-9]+))((e|E)(\-|\+)?[0-9]+)?)?(,((\-|\+)?(([0-9]+(\.[0-9]+)?)|(\.[0-9]+))((e|E)(\-|\+)?[0-9]+)?)?(,((\-|\+)?(([0-9]+(\.[0-9]+)?)|(\.[0-9]+))((e|E)(\-|\+)?[0-9]+)?))?)?)?)*";

    /// <summary>
    /// Fixes the XPS problems it encounters and knows about
    /// </summary>
    public static void FixXps(string filePath)
    {
        // first we'll load the XPS file
        using (var currentPackage = Package.Open(filePath, FileMode.Open))
        {
            var pageUri = new Uri("/Documents/1/Pages/1.fpage", UriKind.Relative);

            // check that the file we'll modify exists
            if (!currentPackage.PartExists(pageUri))
            {
                throw new Exception(string.Format("Unable to find first page in XPS {0} - unable to fix this XPS !", filePath));
            }

            // assume the broken part is in the first page
            var firstPage = currentPackage.GetPart(pageUri);
            var relationships = firstPage.GetRelationships();
            var pageContent = XDocument.Load(firstPage.GetStream());

            // then we'll look up each glyph and check if their "Indices" property is valid
            XNamespace ns = pageContent.Root.GetDefaultNamespace();
            var glyphs = (from g in pageContent.Descendants(ns + "Glyphs")
                          where g.Attribute("Indices") != null
                          select g).ToList();
            for (var i = 0; i < glyphs.Count(); i ++)
            {
                glyphs[i] = FixGlyph(glyphs[i]);
            }

            // remove the current (corrupted) file from the package
            currentPackage.DeletePart(pageUri);

            // add the new (shiny) file to the package
            var newPage = currentPackage.CreatePart(pageUri, "application/vnd.ms-package.xps-fixedpage+xml", CompressionOption.NotCompressed);
            using (var ms = new MemoryStream())
            {
                // we need to remove XML declaration, so we need to use the XmlWriter
                var settings = new XmlWriterSettings();
                settings.Indent = false;
                settings.NewLineChars = string.Empty;
                settings.NewLineHandling = NewLineHandling.Replace;
                settings.OmitXmlDeclaration = true;
                using (var xw = XmlWriter.Create(ms, settings))
                {
                    pageContent.WriteTo(xw);
                }

                ms.Seek(0, SeekOrigin.Begin);
                CopyStream(newPage.GetStream(), ms);
            }

            // now we need to re-create the relationships between the Page file and the fonts
            foreach (var relation in relationships)
            {
                newPage.CreateRelationship(relation.TargetUri, relation.TargetMode, relation.RelationshipType, relation.Id);
            }
        }
    }

    /// <summary>
    /// Tries to load the XPS, and returns false if it fails
    /// </summary>
    public static bool IsValidXps(string filePath)
    {
        try
        {
            using (var xpsOld = new XpsDocument(filePath, FileAccess.Read))
            {
                var unused = xpsOld.GetFixedDocumentSequence();
            }

            return true;
        }
        catch (System.Windows.Markup.XamlParseException)
        {
            return false;
        }
    }

    /// <summary>
    /// Writes the whole content of a stream into another
    /// </summary>
    /// <remarks>
    /// http://stackru.com/a/18885954/2354542
    /// </remarks>
    private static void CopyStream(Stream target, Stream source)
    {
        const int bufSize = 0x1000;
        byte[] buf = new byte[bufSize];
        int bytesRead = 0;
        while ((bytesRead = source.Read(buf, 0, bufSize)) > 0)
        {
            target.Write(buf, 0, bytesRead);
        }
    }

    /// <summary>
    /// Fixes the glyph, if necessary
    /// </summary>
    private static XElement FixGlyph(XElement g)
    {
        var matchAttribute = Regex.Match(g.Attribute("Indices").Value, IndicesRegex);
        if (!matchAttribute.Success)
        {
            return g;
        }

        var hasProblem = false;
        foreach (var token in matchAttribute.Value.Split(";".ToCharArray()))
        {
            if (token == ",")
            {
                hasProblem = true;
                break;
            }
        }

        if (hasProblem)
        {
            // the Indices attribute is not well-formed: let's try to fix the one(s) that are wrong
            var fixedTokens = new List<string>();
            foreach (var token in g.Attribute("Indices").Value.Split(";".ToCharArray()))
            {
                var newToken = token;
                var matchToken = Regex.Match(token, @",(-\d+)");
                if (matchToken.Success) // negative number, yay ! it's not allowed :-(
                {
                    newToken = ",0"; // it should be zero, I believe
                }

                fixedTokens.Add(newToken);
            }

            g.Attribute("Indices").Value = string.Join(";", fixedTokens);
        }

        return g;
    }
}