Differential D19612 Diff 53953 src/bugzillaintegration/duplicatefinderjob.cpp

Changeset View

Standalone View

View Options

src/bugzillaintegration/duplicatefinderjob.cpp

Show All 21 Lines
22#include "drkonqi_debug.h" 22​#include "drkonqi_debug.h"
23 23​
24#include "backtracegenerator.h" 24​#include "backtracegenerator.h"
25#include "parser/backtraceparser.h" 25​#include "parser/backtraceparser.h"
26#include "debuggermanager.h" 26​#include "debuggermanager.h"
27#include "drkonqi.h" 27​#include "drkonqi.h"
28#include "parsebugbacktraces.h" 28​#include "parsebugbacktraces.h"
29 29​
30DuplicateFinderJob::DuplicateFinderJob(const QList<int> &bugIds, BugzillaManager *manager, QObject *parent) 30​DuplicateFinderJob::DuplicateFinderJob(const QList<Bugzilla::Bug::Ptr> &bugs, BugzillaManager *manager, QObject *parent)
31 : KJob(parent), 31​ : KJob(parent),
32 m_manager(manager), 32​ m_manager(manager),
33 m_bugIds(bugIds) 33​ m_bugs(bugs)
34{ 34​{
35 qCDebug(DRKONQI_LOG) << "Possible duplicates:" << m_bugIds; 35​ qCDebug(DRKONQI_LOG) << "Possible duplicates:" << m_bugs.size();
36 connect(m_manager, &BugzillaManager::bugReportFetched, this, &DuplicateFinderJob::slotBugReportFetched); 36​ connect(m_manager, &BugzillaManager::bugReportFetched,
37 connect(m_manager, &BugzillaManager::bugReportError, this, &DuplicateFinderJob::slotBugReportError); 37​ this, &DuplicateFinderJob::slotBugReportFetched);
38​ connect(m_manager, &BugzillaManager::bugReportError,
39​ this, &DuplicateFinderJob::slotError);
40​
41​ connect(m_manager, &BugzillaManager::commentsFetched,
42​ this, &DuplicateFinderJob::slotCommentsFetched);
43​ connect(m_manager, &BugzillaManager::commentsError,
44​ this, &DuplicateFinderJob::slotError);
38} 45​}
39 46​
40DuplicateFinderJob::~DuplicateFinderJob() 47​DuplicateFinderJob::~DuplicateFinderJob()
41{ 48​{
42} 49​}
43 50​
44void DuplicateFinderJob::start() 51​void DuplicateFinderJob::start()
45{ 52​{
46 analyzeNextBug(); 53​ analyzeNextBug();
47} 54​}
48 55​
49DuplicateFinderJob::Result DuplicateFinderJob::result() const 56​DuplicateFinderJob::Result DuplicateFinderJob::result() const
50{ 57​{
51 return m_result; 58​ return m_result;
52} 59​}
53 60​
54void DuplicateFinderJob::analyzeNextBug() 61​void DuplicateFinderJob::analyzeNextBug()
55{ 62​{
56 if (m_bugIds.isEmpty()) { 63​ if (m_bugs.isEmpty()) {
57 emitResult(); 64​ emitResult();
58 return; 65​ return;
59 } 66​ }
60 67​
61 const int bugId = m_bugIds.takeFirst(); 68​ m_bug = m_bugs.takeFirst();
62 qCDebug(DRKONQI_LOG) << "Fetching:" << bugId; 69​ qCDebug(DRKONQI_LOG) << "Fetching:" << m_bug->id();
63 m_manager->fetchBugReport(bugId, this); 70​ m_manager->fetchComments(m_bug, this);
64} 71​}
65 72​
66void DuplicateFinderJob::fetchBug(const QString &bugId) 73​void DuplicateFinderJob::fetchBug(int bugId)
67{ 74​{
68 bool ok; 75​ if (bugId > 0) {
69 const int num = bugId.toInt(&ok); ​
70 if (ok) { ​
71 qCDebug(DRKONQI_LOG) << "Fetching:" << bugId; 76​ qCDebug(DRKONQI_LOG) << "Fetching:" << bugId;
72 m_manager->fetchBugReport(num, this); 77​ m_manager->fetchBugReport(bugId, this);
73 } else { 78​ } else {
74 qCDebug(DRKONQI_LOG) << "Bug id not valid:" << bugId; 79​ qCDebug(DRKONQI_LOG) << "Bug id not valid:" << bugId;
75 analyzeNextBug(); 80​ analyzeNextBug();
76 } 81​ }
77} 82​}
78 83​
79void DuplicateFinderJob::slotBugReportFetched(const BugReport &bug, QObject *owner) 84​void DuplicateFinderJob::slotBugReportFetched(const Bugzilla::Bug::Ptr &bug, QObject *owner)
80{ 85​{
81 if (this != owner) { 86​ if (this != owner) {
82 return; 87​ return;
83 } 88​ }
84 89​
85 ParseBugBacktraces parse(bug, this); 90​#warning kinda code dupe from analyzeNextBug
91​ m_bug = bug;
92​ qCDebug(DRKONQI_LOG) << "Fetching:" << m_bug->id();
93​ m_manager->fetchComments(m_bug, this);
94​}
95​
96​void DuplicateFinderJob::slotCommentsFetched(const QList<Bugzilla::Comment::Ptr> &comments, QObject *owner)
97​{
98​ if (this != owner) {
99​ return;
100​ }
101​
102​ // NOTE: we do not hold the comments in our bug object, once they go out
103​ // of scope they are gone again. We have no use for keeping them in memory
104​ // a user might look at 3 out of 20 bugs, and for those we can simply
105​ // request the comments again instead of holding the potentially very large
106​ // comments in memory.
107​
108​ ParseBugBacktraces parse(comments, this);
86 parse.parse(); 109​ parse.parse();
87 110​
88 BacktraceGenerator *btGenerator = DrKonqi::debuggerManager()->backtraceGenerator(); 111​ BacktraceGenerator *btGenerator = DrKonqi::debuggerManager()->backtraceGenerator();
89 const ParseBugBacktraces::DuplicateRating rating = parse.findDuplicate(btGenerator->parser()->parsedBacktraceLines()); 112​ const ParseBugBacktraces::DuplicateRating rating = parse.findDuplicate(btGenerator->parser()->parsedBacktraceLines());
90 qCDebug(DRKONQI_LOG) << "Duplicate rating:" << rating; 113​ qCDebug(DRKONQI_LOG) << "Duplicate rating:" << rating;
91 114​
92 //TODO handle more cases here 115​ //TODO handle more cases here
93 if (rating != ParseBugBacktraces::PerfectDuplicate) { 116​ if (rating != ParseBugBacktraces::PerfectDuplicate) {
94 qCDebug(DRKONQI_LOG) << "Bug" << bug.bugNumber() << "most likely not a duplicate:" << rating; 117​ qCDebug(DRKONQI_LOG) << "Bug" << m_bug->id() << "most likely not a duplicate:" << rating;
95 analyzeNextBug(); 118​ analyzeNextBug();
96 return; 119​ return;
97 } 120​ }
98 121​
122​ bool unknownStatus = (m_bug->status() == Bugzilla::Bug::Status::Unknown);
123​ bool unknownResolution = (m_bug->resolution() == Bugzilla::Bug::Resolution::Unknown);
124​
99 //The Bug is a duplicate, now find out the status and resolution of the existing report 125​ //The Bug is a duplicate, now find out the status and resolution of the existing report
100 if (bug.resolutionValue() == BugReport::Duplicate) { 126​ if (m_bug->resolution() == Bugzilla::Bug::Resolution::DUPLICATE) {
101 qCDebug(DRKONQI_LOG) << "Found duplicate is a duplicate itself."; 127​ qCDebug(DRKONQI_LOG) << "Found duplicate is a duplicate itself.";
102 if (!m_result.duplicate) { 128​ if (!m_result.duplicate) {
103 m_result.duplicate = bug.bugNumberAsInt(); 129​ m_result.duplicate = m_bug->id();
104 } 130​ }
105 fetchBug(bug.markedAsDuplicateOf()); 131​ fetchBug(m_bug->dupe_of());
106 } else if ((bug.statusValue() == BugReport::UnknownStatus) || (bug.resolutionValue() == BugReport::UnknownResolution)) { 132​#warning fixme can unknown happen even or could it before how does this work now
133​ } else if (unknownStatus || unknownResolution) {
107 qCDebug(DRKONQI_LOG) << "Either the status or the resolution is unknown."; 134​ qCDebug(DRKONQI_LOG) << "Either the status or the resolution is unknown.";
108 qCDebug(DRKONQI_LOG) << "Status \"" << bug.bugStatus() << "\" known:" << (bug.statusValue() != BugReport::UnknownStatus); 135​ qCDebug(DRKONQI_LOG) << "Status \"" << m_bug->status() << "\" known:" << !unknownStatus;
109 qCDebug(DRKONQI_LOG) << "Resolution \"" << bug.resolution() << "\" known:" << (bug.resolutionValue() != BugReport::UnknownResolution); 136​ qCDebug(DRKONQI_LOG) << "Resolution \"" << m_bug->resolution() << "\" known:" << !unknownResolution;
110 analyzeNextBug(); 137​ analyzeNextBug();
111 } else { 138​ } else {
112 if (!m_result.duplicate) { 139​ if (!m_result.duplicate) {
113 m_result.duplicate = bug.bugNumberAsInt(); 140​ m_result.duplicate = m_bug->id();
114 } 141​ }
115 m_result.parentDuplicate = bug.bugNumberAsInt(); 142​ m_result.parentDuplicate = m_bug->id();
116 m_result.status = bug.statusValue(); 143​ m_result.status = m_bug->status();
117 m_result.resolution = bug.resolutionValue(); 144​ m_result.resolution = m_bug->resolution();
118 qCDebug(DRKONQI_LOG) << "Found duplicate information (id/status/resolution):" << bug.bugNumber() << bug.bugStatus() << bug.resolution(); 145​ qCDebug(DRKONQI_LOG) << "Found duplicate information (id/status/resolution):"
146​ << m_bug->id() << m_bug->status() << m_bug->resolution();
119 emitResult(); 147​ emitResult();
120 } 148​ }
121} 149​}
122 150​
123void DuplicateFinderJob::slotBugReportError(const QString &message, QObject *owner) 151​void DuplicateFinderJob::slotError(const QString &message, QObject *owner)
124{ 152​{
125 if (this != owner) { 153​ if (this != owner) {
126 return; 154​ return;
127 } 155​ }
128 qCDebug(DRKONQI_LOG) << "Error fetching bug:" << message; 156​ qCDebug(DRKONQI_LOG) << "Error fetching bug:" << message;
129 analyzeNextBug(); 157​ analyzeNextBug();
130} 158​}
159​