qulice icon indicating copy to clipboard operation
qulice copied to clipboard

Qulice fails with unintuitive message on Log4j2 configuration

Open mkordas opened this issue 9 years ago • 25 comments

When Qulice tries to validate XML without schema attached, e.g. Log4j2 configuration:

<?xml version="1.0" encoding="UTF-8"?>
<Configuration>
    <Appenders>
        <Console name="CONSOLE" target="SYSTEM_OUT">
            <PatternLayout pattern="[%p] %t %c: %m%n"/>
        </Console>
    </Appenders>
    <Loggers>
        <Root level="WARN">
            <AppenderRef ref="CONSOLE"/>
        </Root>
    </Loggers>
</Configuration>

it fails with the following message:

[WARNING] 1 XML validation error(s):
  -1:-1: cvc-elt.1: Cannot find the declaration of element 'Configuration'.

Qulice should not fail during analysis of XML files without schema.

mkordas avatar Jan 23 '16 00:01 mkordas

@mkordas well, it is there to validate the document against a schema (and style), so if there is none it fails, although it needs a better message

krzyk avatar Jan 23 '16 07:01 krzyk

@krzyk My point is that Qulice should not start XML schema validation if it doesn't have schema for that file, because there might be valid XML files like non-strict Log4j2 configuration for which schema doesn't even exist.

mkordas avatar Jan 23 '16 11:01 mkordas

@davvd valid bug

krzyk avatar Jan 23 '16 13:01 krzyk

@davvd this is postponed

krzyk avatar Jan 23 '16 13:01 krzyk

@davvd valid bug

@krzyk tag "bug" added

davvd avatar Jan 25 '16 07:01 davvd

@davvd this is postponed

@krzyk OK, I put "postponed" label here

davvd avatar Jan 25 '16 10:01 davvd

@davvd this is postponed

@krzyk someone else will help in this task, no problem at all

davvd avatar Jan 25 '16 10:01 davvd

@davvd assing @jrdalpra

krzyk avatar Jan 26 '16 07:01 krzyk

@mkordas thanks for the report, I topped your acc for 15 mins, payment ID 56a759c9171e7b555200025a

davvd avatar Jan 26 '16 11:01 davvd

@krzyk Still not assigned to me.

jrdalpra avatar Feb 02 '16 00:02 jrdalpra

@davvd assign @jrdalpra please

krzyk avatar Feb 02 '16 07:02 krzyk

@davvd this is not postponed

krzyk avatar Feb 02 '16 07:02 krzyk

@davvd assign @jrdalpra please

@krzyk yep, done. @jrdalpra it is your task now

davvd avatar Feb 04 '16 07:02 davvd

@davvd this is not postponed

@krzyk thanks, I removed the "postponed" tag

davvd avatar Feb 04 '16 07:02 davvd

@jrdalpra how is the work going, need any help?

krzyk avatar Feb 07 '16 10:02 krzyk

@krzyk I was working on another issue. I'll work on this one tomorrow. Thanks.

jrdalpra avatar Feb 07 '16 14:02 jrdalpra

@jrdalpra there is a more important issue (#676) could I switch you to that one?

krzyk avatar Feb 07 '16 15:02 krzyk

@davvd this is postponed

krzyk avatar Feb 07 '16 16:02 krzyk

@krzyk sure. Give to me.

jrdalpra avatar Feb 07 '16 20:02 jrdalpra

@davvd this is postponed

@krzyk got it, "postponed" label here

davvd avatar Feb 08 '16 09:02 davvd

@davvd this is postponed

@krzyk right, I will find someone else, no problem

davvd avatar Feb 08 '16 09:02 davvd

@davvd https://github.com/davvd @krzyk https://github.com/krzyk Is the task mine or not? Thanks.

José Volmei Dal Prá Junior - (49) 9112 9359 Bacharel em Ciência da Computação - UnoChapecó Pós-Graduado em Desenvolvimento de Software Com Java - Unoesc Xxe Consultor Java e TotalCross https://github.com/jrdalpra http://www.kombo.com.br/meucurriculo/jrdalpra http://www.linkedin.com/in/jrdalpra msn: [email protected] skype: jrdalpra e-mail: [email protected]

jrdalpra avatar Feb 08 '16 14:02 jrdalpra

@jrdalpra this one no, it's a minor one. Let's wait for new assignment

krzyk avatar Feb 08 '16 14:02 krzyk

@jrdalpra

I was working on another issue. I'll work on this one tomorrow. Thanks.

Please update your comment with a name tag.

mkordas avatar Feb 09 '16 21:02 mkordas

The bug is still present... Are there any updates on when it will be fixed? v. 0.17.1

driver733 avatar May 21 '17 04:05 driver733