This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 225742 - Change source lines parsing
Summary: Change source lines parsing
Status: RESOLVED FIXED
Alias: None
Product: qa
Classification: Unclassified
Component: Dashboard (show other bugs)
Version: 7.3
Hardware: PC Linux
: P1 normal (vote)
Assignee: Vladimir Riha
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-05 10:02 UTC by Vladimir Riha
Modified: 2013-02-07 15:23 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir Riha 2013-02-05 10:02:05 UTC
Right now incoming source lines data (diff & total) are mapped to component_id and save to database. Change it so module name is saved to the database instead of component_id

- change view
- change input processing
- change schema
- change sources (developers), team pages etc.
Comment 1 Vladimir Riha 2013-02-06 10:41:58 UTC
First part (total source lines) is hopefully done, what's left:

- revision data (added...) - saving & retrieving
- some workaround for NB73 static archive - so changes made in 320 are not used yet, but almost ready to be

As a side effect, I tried to optimize used DB queries and it seems to be noticeable faster :)
Comment 2 Vladimir Riha 2013-02-06 12:28:57 UTC
(In reply to comment #1)
> - revision data (added...) - saving & retrieving

Should be done by 322, changes are again not used yet

> - some workaround for NB73 static archive

An the same goes for team pages... I think I just download all files manually from browser (so with fetched data) and manually replace original files on server. Otherwise I'd have to do some very ugly workarounds in server to preserve both kind of data retrieving.
Comment 3 Vladimir Riha 2013-02-06 13:50:26 UTC
(In reply to comment #2)
> An the same goes for team pages... 

I manually modified those files. For today I can't do anything else, tomorrow once dashboard will have the newly saved data, I'll switch total source lines to the modified methods.
Comment 4 Vladimir Riha 2013-02-07 13:59:07 UTC
Some minor typos were pushed in #333. One problem is that Source Lines per IDE are not working :/ I have no idea why, I put logging at the very beginning of the particular php file and yet no logs are produced... I'll search more
Comment 5 Vladimir Riha 2013-02-07 15:23:22 UTC
Fixed by 337. Now it all seems to be OK, numbers for 8.0 are slightly different due to ongoing development but still similar to 7.3, which is imo expected. I can't test source lines diff as data will be accepted after 7.3 FCS, but it should be OK as well (now the refactored methods are being used anyway, only returns empty results)