Merge pull request #5401 from tsiegleauq/assignment-poll-chart-abstain
Hide abstain bar in assignment polls chart
This commit is contained in:
commit
ff9125fb9f
@ -309,7 +309,11 @@ export abstract class PollService {
|
||||
public generateChartData(poll: PollData | ViewBasePoll): ChartData {
|
||||
const fields = this.getPollDataFields(poll);
|
||||
|
||||
const data: ChartData = fields.map(key => {
|
||||
const data: ChartData = fields
|
||||
.filter(key => {
|
||||
return this.getPollDataFieldsByPercentBase(poll).includes(key);
|
||||
})
|
||||
.map(key => {
|
||||
return {
|
||||
data: this.getResultFromPoll(poll, key),
|
||||
label: key.toUpperCase(),
|
||||
@ -324,34 +328,36 @@ export abstract class PollService {
|
||||
}
|
||||
|
||||
protected getPollDataFields(poll: PollData | ViewBasePoll): CalculablePollKey[] {
|
||||
let fields: CalculablePollKey[];
|
||||
let isAssignment: boolean;
|
||||
|
||||
if (poll instanceof ViewBasePoll) {
|
||||
isAssignment = poll.pollClassType === 'assignment';
|
||||
} else {
|
||||
isAssignment = Object.keys(poll.options[0]).includes('user');
|
||||
const isAssignment: boolean = (poll as ViewBasePoll).pollClassType === 'assignment';
|
||||
return isAssignment ? this.getPollDataFieldsByMethod(poll) : this.getPollDataFieldsByPercentBase(poll);
|
||||
}
|
||||
|
||||
if (isAssignment) {
|
||||
if (poll.pollmethod === AssignmentPollMethod.YNA) {
|
||||
fields = ['yes', 'no', 'abstain'];
|
||||
} else if (poll.pollmethod === AssignmentPollMethod.YN) {
|
||||
fields = ['yes', 'no'];
|
||||
} else {
|
||||
fields = ['yes'];
|
||||
private getPollDataFieldsByMethod(poll: PollData | ViewBasePoll): CalculablePollKey[] {
|
||||
switch (poll.pollmethod) {
|
||||
case AssignmentPollMethod.YNA: {
|
||||
return ['yes', 'no', 'abstain'];
|
||||
}
|
||||
case AssignmentPollMethod.YN: {
|
||||
return ['yes', 'no'];
|
||||
}
|
||||
default: {
|
||||
return ['yes'];
|
||||
}
|
||||
} else {
|
||||
if (poll.onehundred_percent_base === PercentBase.YN) {
|
||||
fields = ['yes', 'no'];
|
||||
} else if (poll.onehundred_percent_base === PercentBase.Cast) {
|
||||
fields = ['yes', 'no', 'abstain', 'votesinvalid'];
|
||||
} else {
|
||||
fields = ['yes', 'no', 'abstain'];
|
||||
}
|
||||
}
|
||||
|
||||
return fields;
|
||||
private getPollDataFieldsByPercentBase(poll: PollData | ViewBasePoll): CalculablePollKey[] {
|
||||
switch (poll.onehundred_percent_base) {
|
||||
case PercentBase.YN: {
|
||||
return ['yes', 'no'];
|
||||
}
|
||||
case PercentBase.Cast: {
|
||||
return ['yes', 'no', 'abstain', 'votesinvalid'];
|
||||
}
|
||||
default: {
|
||||
return ['yes', 'no', 'abstain'];
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
|
Loading…
Reference in New Issue
Block a user