This is the codeAbility Sharing Platform! Learn more about the
codeAbility Sharing Platform
.
Skip to content
GitLab
Explore
Sign in
Register
Primary navigation
Search or go to…
Project
codeAbility Sharing Platform
Manage
Activity
Members
Labels
Plan
Issues
43
Issue boards
Milestones
Wiki
Code
Merge requests
1
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Deploy
Releases
Analyze
Contributor analytics
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
development
sharing
codeAbility Sharing Platform
Commits
5961660e
Commit
5961660e
authored
2 years ago
by
Michael Breu
Browse files
Options
Downloads
Patches
Plain Diff
refactoring duplicated code
parent
9e01f809
Branches
Branches containing commit
3 merge requests
!188
Merging Peer Reviewing et. al to Master
,
!181
Resolve "Prüfen des Caches der Review Badge Statistiken"
,
!164
211 peer reviewing functionality
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
src/main/webapp/app/layouts/navbar/navbar-badge.component.spec.ts
+8
-9
8 additions, 9 deletions
.../webapp/app/layouts/navbar/navbar-badge.component.spec.ts
src/main/webapp/app/layouts/navbar/navbar.component.ts
+30
-68
30 additions, 68 deletions
src/main/webapp/app/layouts/navbar/navbar.component.ts
with
38 additions
and
77 deletions
src/main/webapp/app/layouts/navbar/navbar-badge.component.spec.ts
+
8
−
9
View file @
5961660e
...
...
@@ -71,9 +71,8 @@ describe('ReviewMenuBadgeComponent Component', () => {
});
testRequest
.
flush
(
stats
);
expect
(
comp
.
getReviewActionCount
()).
toEqual
(
stats
.
reviewInProgress
+
stats
.
reviewImproved
);
// expect(comp.getReviewActionLabel()).toEqual("2 reviews in progress or improvement requested.");
expect
(
comp
.
getReviewActionLabel
()).
toEqual
(
'
1 item(s) for review.
\n
2 improved item(s) for review
'
);
expect
(
comp
.
getCount
()).
toEqual
(
stats
.
reviewInProgress
+
stats
.
reviewImproved
);
expect
(
comp
.
getLabel
()).
toEqual
(
'
1 item(s) for review.
\n
2 improved item(s) for review
'
);
});
});
...
...
@@ -117,8 +116,8 @@ describe('ReviewRequestedMenuBadge Component', () => {
});
testRequest
.
flush
(
stats
);
expect
(
comp
.
get
ReviewRequested
Count
()).
toEqual
(
stats
.
reviewsRequestedByUser
);
expect
(
comp
.
get
ReviewRequested
Label
()).
toEqual
(
'
5 reviews request, but not yet assigned.
'
);
expect
(
comp
.
getCount
()).
toEqual
(
stats
.
reviewsRequestedByUser
);
expect
(
comp
.
getLabel
()).
toEqual
(
'
5 reviews request, but not yet assigned.
'
);
});
});
...
...
@@ -162,8 +161,8 @@ describe('ImprovementsRequestedMenuBadge Component', () => {
});
testRequest
.
flush
(
stats
);
expect
(
comp
.
get
ReviewRequested
Count
()).
toEqual
(
7
);
expect
(
comp
.
get
ReviewRquested
Label
()).
toEqual
(
'
7 review(s) resulted in an improvements request
'
);
expect
(
comp
.
getCount
()).
toEqual
(
7
);
expect
(
comp
.
getLabel
()).
toEqual
(
'
7 review(s) resulted in an improvements request
'
);
});
});
...
...
@@ -207,7 +206,7 @@ describe('BadgeAwardedMenuBadge Component', () => {
});
testRequest
.
flush
(
stats
);
expect
(
comp
.
get
Badge
Count
()).
toEqual
(
10
);
expect
(
comp
.
get
ReviewRquested
Label
()).
toEqual
(
'
10 badge(s) awarded
'
);
expect
(
comp
.
getCount
()).
toEqual
(
10
);
expect
(
comp
.
getLabel
()).
toEqual
(
'
10 badge(s) awarded
'
);
});
});
This diff is collapsed.
Click to expand it.
src/main/webapp/app/layouts/navbar/navbar.component.ts
+
30
−
68
View file @
5961660e
import
{
Location
}
from
'
@angular/common
'
;
import
{
Component
,
Input
,
OnInit
}
from
'
@angular/core
'
;
import
{
Component
,
Directive
,
Input
,
OnInit
}
from
'
@angular/core
'
;
import
{
FormBuilder
,
FormGroup
}
from
'
@angular/forms
'
;
import
{
Router
}
from
'
@angular/router
'
;
import
{
TranslateService
}
from
'
@ngx-translate/core
'
;
...
...
@@ -208,18 +208,8 @@ export class MenuItemBadgeComponent {
@
Input
()
public
label
:
string
|
undefined
;
}
@
Component
({
selector
:
'
jhi-reviewMenuBadge
'
,
template
:
`<jhi-menuItemBadge
*ngIf="getReviewActionCount()"
[text]="getReviewActionCount()"
badgeStyle="info"
[label]="getReviewActionLabel()"
>
</jhi-menuItemBadge> `
,
styles
:
[
''
],
})
export
class
ReviewMenuBadgeComponent
implements
OnInit
{
@
Directive
()
abstract
class
ReviewBadgeComponent
implements
OnInit
{
statistics
:
ReviewStatisticsDTO
|
undefined
;
constructor
(
private
reviewManagementService
:
ReviewManagementService
)
{}
...
...
@@ -230,13 +220,26 @@ export class ReviewMenuBadgeComponent implements OnInit {
});
}
getReviewActionCount
():
number
{
abstract
getCount
():
number
;
abstract
getLabel
():
string
|
undefined
;
}
@
Component
({
selector
:
'
jhi-reviewMenuBadge
'
,
template
:
`<jhi-menuItemBadge *ngIf="getCount()" [text]="getCount()" badgeStyle="info" [label]="getLabel()"> </jhi-menuItemBadge> `
,
styles
:
[
''
],
})
export
class
ReviewMenuBadgeComponent
extends
ReviewBadgeComponent
{
statistics
:
ReviewStatisticsDTO
|
undefined
;
getCount
():
number
{
if
(
this
.
statistics
)
{
return
this
.
statistics
.
reviewInProgress
+
this
.
statistics
.
reviewImproved
;
}
return
0
;
}
get
ReviewAction
Label
():
string
|
undefined
{
getLabel
():
string
|
undefined
{
let
info
=
''
;
let
trenn
=
''
;
if
(
this
.
statistics
?.
reviewInProgress
)
{
...
...
@@ -253,33 +256,19 @@ export class ReviewMenuBadgeComponent implements OnInit {
@
Component
({
selector
:
'
jhi-reviewRequestedMenuBadge
'
,
template
:
`<jhi-menuItemBadge
*ngIf="getReviewRequestedCount()"
[text]="getReviewRequestedCount()"
badgeStyle="info"
[label]="getReviewRequestedLabel()"
>
</jhi-menuItemBadge> `
,
template
:
`<jhi-menuItemBadge *ngIf="getCount()" [text]="getCount()" badgeStyle="info" [label]="getLabel()"> </jhi-menuItemBadge> `
,
styles
:
[
''
],
})
export
class
ReviewRequestedMenuBadgeComponent
implem
en
t
s
OnIni
t
{
export
class
ReviewRequestedMenuBadgeComponent
ext
en
d
s
ReviewBadgeComponen
t
{
statistics
:
ReviewStatisticsDTO
|
undefined
;
constructor
(
private
reviewManagementService
:
ReviewManagementService
)
{}
ngOnInit
():
void
{
this
.
reviewManagementService
.
getReviewStatistics
().
subscribe
((
res
:
ReviewStatisticsDTO
)
=>
{
this
.
statistics
=
res
;
});
}
getReviewRequestedCount
():
number
{
getCount
():
number
{
if
(
this
.
statistics
)
{
return
this
.
statistics
.
reviewsRequestedByUser
;
}
return
0
;
}
get
ReviewRequested
Label
():
string
|
undefined
{
getLabel
():
string
|
undefined
{
let
info
=
''
;
if
(
this
.
statistics
?.
reviewsRequestedByUser
)
{
info
=
`
${
this
.
statistics
.
reviewsRequestedByUser
}
reviews request, but not yet assigned.`
;
...
...
@@ -290,33 +279,17 @@ export class ReviewRequestedMenuBadgeComponent implements OnInit {
@
Component
({
selector
:
'
jhi-reviewImprovementsRequestedMenuBadge
'
,
template
:
`<jhi-menuItemBadge
*ngIf="getReviewRequestedCount()"
[text]="getReviewRequestedCount()"
badgeStyle="danger"
[label]="getReviewRquestedLabel()"
>
</jhi-menuItemBadge> `
,
template
:
`<jhi-menuItemBadge *ngIf="getCount()" [text]="getCount()" badgeStyle="danger" [label]="getLabel()"> </jhi-menuItemBadge> `
,
styles
:
[
''
],
})
export
class
ImprovementsRequestedMenuBadgeComponent
implements
OnInit
{
statistics
:
ReviewStatisticsDTO
|
undefined
;
constructor
(
private
reviewManagementService
:
ReviewManagementService
)
{}
ngOnInit
():
void
{
this
.
reviewManagementService
.
getReviewStatistics
().
subscribe
((
res
:
ReviewStatisticsDTO
)
=>
{
this
.
statistics
=
res
;
});
}
getReviewRequestedCount
():
number
{
export
class
ImprovementsRequestedMenuBadgeComponent
extends
ReviewBadgeComponent
{
getCount
():
number
{
if
(
this
.
statistics
)
{
return
this
.
statistics
.
reviewImprovementRequestedByUser
;
}
return
0
;
}
get
ReviewRquested
Label
():
string
|
undefined
{
getLabel
():
string
|
undefined
{
let
info
=
''
;
if
(
this
.
statistics
?.
reviewImprovementRequestedByUser
)
{
info
=
`
${
this
.
statistics
.
reviewImprovementRequestedByUser
}
review(s) resulted in an improvements request`
;
...
...
@@ -327,28 +300,17 @@ export class ImprovementsRequestedMenuBadgeComponent implements OnInit {
@
Component
({
selector
:
'
jhi-badgesRewardedMenuBadge
'
,
template
:
`<jhi-menuItemBadge *ngIf="getBadgeCount()" [text]="getBadgeCount()" badgeStyle="success" [label]="getReviewRquestedLabel()">
</jhi-menuItemBadge> `
,
template
:
`<jhi-menuItemBadge *ngIf="getCount()" [text]="getCount()" badgeStyle="success" [label]="getLabel()"> </jhi-menuItemBadge> `
,
styles
:
[
''
],
})
export
class
BadgeAwardedMenuBadgeComponent
implements
OnInit
{
statistics
:
ReviewStatisticsDTO
|
undefined
;
constructor
(
private
reviewManagementService
:
ReviewManagementService
)
{}
ngOnInit
():
void
{
this
.
reviewManagementService
.
getReviewStatistics
().
subscribe
((
res
:
ReviewStatisticsDTO
)
=>
{
this
.
statistics
=
res
;
});
}
getBadgeCount
():
number
{
export
class
BadgeAwardedMenuBadgeComponent
extends
ReviewBadgeComponent
{
getCount
():
number
{
if
(
this
.
statistics
)
{
return
this
.
statistics
.
badgesRewarded
;
}
return
0
;
}
get
ReviewRquested
Label
():
string
|
undefined
{
getLabel
():
string
|
undefined
{
let
info
=
''
;
if
(
this
.
statistics
?.
badgesRewarded
)
{
info
=
`
${
this
.
statistics
.
badgesRewarded
}
badge(s) awarded`
;
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment