Skip to content

Conversation

@icewind1991
Copy link
Member

Make sure that the option is what we expect.

Probably caused by a not 100% valid info.xml, not sure which one though, found the issue in the logs of a customer.

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Feb 7, 2018
@icewind1991 icewind1991 added this to the Nextcloud 14 milestone Feb 7, 2018
@MorrisJobke
Copy link
Member

bildschirmfoto 2018-02-07 um 14 24 43

I had this as well for the zimbra app: https://github.com/ZeXtras/zimbra-drive

The sentry log looks like the $options should not be iterated on but is the actual $option. 🤔

@MorrisJobke
Copy link
Member

This is a working entry in the info.xml:

       <description>Authenticate Users by SQL</description>

This is a broken one:

    <description lang="en">
        <![CDATA[Add Nextcloud / ownCloud as Briefcase in Zimbra with the full features of both applications.]]></description>

So maybe something is wrong in the parser itself.

@icewind1991
Copy link
Member Author

Superseeded by #9159

@icewind1991 icewind1991 deleted the verify-l10n-option branch April 12, 2018 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants