Skip to content
This repository has been archived by the owner on Aug 24, 2019. It is now read-only.

PersistanceMode dropped by sonarqube 7.0 #58

Merged
merged 1 commit into from
Jul 16, 2018

Conversation

Pavel87
Copy link

@Pavel87 Pavel87 commented Mar 8, 2018

PersistanceMode class was removed from Sonarqube, this commit solves the following error.

ERROR: Error during SonarQube Scanner execution
java.lang.NoClassDefFoundError: org/sonar/api/measures/PersistenceMode
at org.sonar.plugins.objectivec.complexity.LizardReportParser.buildMeasureList(LizardReportParser.java:159)
at org.sonar.plugins.objectivec.complexity.LizardReportParser.addComplexityFileMeasures(LizardReportParser.java:140)
at org.sonar.plugins.objectivec.complexity.LizardReportParser.parseFile(LizardReportParser.java:109)
at org.sonar.plugins.objectivec.complexity.LizardReportParser.parseReport(LizardReportParser.java:77)
at org.sonar.plugins.objectivec.complexity.LizardSensor.parseReportsIn(LizardSensor.java:88)
at org.sonar.plugins.objectivec.complexity.LizardSensor.analyse(LizardSensor.java:73)

@JackLeo
Copy link

JackLeo commented Mar 9, 2018

Tested this in a live system. No more breakages

@Pavel87 Pavel87 closed this Mar 9, 2018
@raatiniemi
Copy link

@Pavel87 why was this PR closed? From what I can tell, it wasn't merged, only approved, or am I missing something?

@Pavel87
Copy link
Author

Pavel87 commented Mar 12, 2018

Sorry for confusion, I thought it was merged already.

@raatiniemi
Copy link

No problem, the user-interface isn't always that clear. Hopefully this gets merged soon, I've done similar changes in #60 but I'd be happy to rebase if/when this gets merged.

@Basca
Copy link

Basca commented May 30, 2018

@Throll As you were also active on sonar-swift, could you get this merged? For all the people using sonar 7.x

@meltenc
Copy link

meltenc commented Jun 2, 2018

@Basca I will check and merge this as soon as i can

@JackLeo
Copy link

JackLeo commented Jul 16, 2018

Was it merged?

@meltenc meltenc merged commit e8b0e17 into Backelite:develop Jul 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants