Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
34628e7
WIP add marker flags control to tasks
vehikl-jacare Nov 27, 2020
62d373c
WIP: add task marker options in advanced inspector
vehikl-jacare Nov 30, 2020
4efa251
add custom data transform for task
vehikl-jacare Nov 30, 2020
6f7044f
set some default values for markerFlags
vehikl-jacare Nov 30, 2020
cdaadd2
remove compensation icon if isForCompensation is false
vehikl-jacare Nov 30, 2020
f26c5d8
cleanup compensation icon logic
vehikl-jacare Nov 30, 2020
125e4df
refactor watcher for compensation
vehikl-jacare Nov 30, 2020
802294c
get standard loop characteristic to show up
vehikl-jacare Nov 30, 2020
b44cda4
unset loop icon
vehikl-jacare Nov 30, 2020
d2c5440
properly remove marker
vehikl-jacare Nov 30, 2020
b433ab0
sort the bottom center markers alphabetically
vehikl-jacare Nov 30, 2020
41f76d0
assert that bottom centre markers get sorted
vehikl-jacare Nov 30, 2020
86edac8
did do the todo
vehikl-jacare Nov 30, 2020
c48a2d9
set the MarkerFlags radio values from the bpmn
vehikl-jacare Nov 30, 2020
9a7cfa8
WIP: add e2e tests for adding task marker flags
vehikl-jacare Nov 30, 2020
1192d00
add tests for task markers
vehikl-jacare Dec 1, 2020
c2f4cc0
Fix spacing
vehikl-jacare Dec 1, 2020
99aafdf
reword tests
vehikl-jacare Dec 1, 2020
b5ee459
fix failing test
vehikl-jacare Dec 1, 2020
2de44f8
add failing test
vehikl-jacare Dec 2, 2020
e105d32
space
vehikl-jacare Dec 2, 2020
f4ff120
remove .only
vehikl-jacare Dec 2, 2020
cc0061c
fix watcher bug
vehikl-jacare Dec 3, 2020
83a26d1
fix setting props so they are reactive
vehikl-jacare Dec 3, 2020
690ebcc
Merge branch 'develop' into 1271-add-loop-characteristic-toggles
vehikl-jacare Dec 3, 2020
7e3aae9
update data input and output association ids
vehikl-jacare Dec 3, 2020
536a750
lint spacinig
vehikl-jacare Dec 3, 2020
c9258af
fix null definitiion reference error
vehikl-jacare Dec 3, 2020
1e48d3f
Update ci test spec
vehikl-jacare Dec 3, 2020
1eb4345
add test for no validation errors
vehikl-jacare Dec 3, 2020
320767c
update assertion as timer and name accordions are now linked
vehikl-jacare Dec 3, 2020
a76485f
Revert "update assertion as timer and name accordions are now linked"
vehikl-jacare Dec 4, 2020
c778035
rename inspector accordion names to be unique
vehikl-jacare Dec 4, 2020
2ee7f70
fix unsetting definition value
vehikl-jacare Dec 4, 2020
c2f7f7e
add test for unsetting loop characteristics
vehikl-jacare Dec 4, 2020
99281af
rename
vehikl-jacare Dec 4, 2020
5af7081
refactor method so early return works without skipping other updates
vehikl-jacare Dec 7, 2020
d7fffb3
early return to remove one level of if block
vehikl-jacare Dec 7, 2020
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"check-peer-deps": "npm ls --depth=0",
"posttest": "npm run report:combined",
"test-unit": "JEST_JUNIT_OUTPUT_DIR=test_summary_reports/jest/ vue-cli-service test:unit --ci --runInBand --reporters=default --reporters=jest-junit",
"test-ci": "TZ=UTC vue-cli-service test:e2e --spec tests/e2e/specs/Modeler.spec.js --mode test --headless --browser chrome --reporter mocha-junit-reporter --reporter-options mochaFile=test_summary_reports/mocha/test-results.xml",
"test-ci": "TZ=UTC vue-cli-service test:e2e --mode test --headless --browser chrome -- --reporter mocha-junit-reporter --reporter-options mochaFile=reports/mocha/test-results.xml",
"mkdir:reports": "mkdir reports || true",
"precopy:reports": "npm run mkdir:reports",
"copy:reports": "cp cypress-coverage/coverage-final.json reports/from-cypress.json && cp jest-coverage/coverage-final.json reports/from-jest.json",
Expand Down
16 changes: 9 additions & 7 deletions src/components/inspectors/InspectorPanel.vue
Original file line number Diff line number Diff line change
Expand Up @@ -175,13 +175,15 @@ export default {

const type = this.highlightedNode && this.highlightedNode.type;

this.data = type && this.nodeRegistry[type].inspectorData
? this.nodeRegistry[type].inspectorData(this.highlightedNode)
: Object.entries(this.highlightedNode.definition).reduce((data, [key, value]) => {
data[key] = value;
const defaultDataTransform = (node) => Object.entries(node.definition).reduce((data, [key, value]) => {
data[key] = value;

return data;
}, {});

return data;
}, {});
this.data = type && this.nodeRegistry[type].inspectorData
? this.nodeRegistry[type].inspectorData(this.highlightedNode, defaultDataTransform)
: defaultDataTransform(this.highlightedNode);
},
isSequenceFlow(type) {
return type === sequenceFlowId;
Expand All @@ -193,7 +195,7 @@ export default {
return definition.targetRef.$type === 'bpmn:CallActivity';
},
customInspectorHandler(value) {
return this.nodeRegistry[this.highlightedNode.type].inspectorHandler(value, this.highlightedNode, this.setNodeProp, this.moddle, this.definitions);
return this.nodeRegistry[this.highlightedNode.type].inspectorHandler(value, this.highlightedNode, this.setNodeProp, this.moddle, this.definitions, this.defaultInspectorHandler);
},
processNodeInspectorHandler(value) {
return this.defaultInspectorHandler(omit(value, ['artifacts', 'flowElements', 'laneSets']));
Expand Down
48 changes: 48 additions & 0 deletions src/components/inspectors/MarkerFlags.vue
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
<template>
<div>
<b-form-group label="Loop Characteristics">
<b-form-radio data-test="no_loop" v-model="loopCharacteristics" name="marker-flags-radios" value="no_loop">No loop characteristics</b-form-radio>
<b-form-radio data-test="parallel_mi" v-model="loopCharacteristics" name="marker-flags-radios" value="parallel_mi">Parallel multi-instance</b-form-radio>
<b-form-radio data-test="sequential_mi" v-model="loopCharacteristics" name="marker-flags-radios" value="sequential_mi">Sequential multi-instance</b-form-radio>
<b-form-radio data-test="loop" v-model="loopCharacteristics" name="marker-flags-radios" value="loop">Loop</b-form-radio>
</b-form-group>
<b-form-group>
<b-form-checkbox data-test="for-compensation" v-model="isForCompensation" name="for-compensation">For Compensation</b-form-checkbox>
</b-form-group>
</div>
</template>

<script>
export default {
props: {
value: {
type: Object,
default() {
return {
loopCharacteristics: 'no_loop',
isForCompensation: false,
};
},
},
},
name: 'MarkerFlags',
data() {
return {
loopCharacteristics: this.value.loopCharacteristics,
isForCompensation: this.value.isForCompensation,
};
},
watch: {
loopCharacteristics(newVal) {
this.$emit('input', {loopCharacteristics: newVal});
},
isForCompensation(newVal) {
this.$emit('input', {isForCompensation: newVal});
},
},
};
</script>

<style scoped>

</style>
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
import idConfigSettings from './idConfigSettings';
import DocumentationFormTextArea from './DocumentationFormTextArea';
import MarkerFlags from '@/components/inspectors/MarkerFlags';

export default {
component: 'FormAccordion',
container: true,
config: {
initiallyOpen: false,
label: 'Advanced',
icon: 'cogs',
name: 'advanced-accordion',
},
items: [
{
component: 'FormInput',
config: idConfigSettings,
},
{
component: DocumentationFormTextArea,
config: {
label: 'Description',
name: 'documentation',
},
},
{
component: MarkerFlags,
name: 'taskMarkers',
config: {
label: 'Marker Flags',
name: 'markerFlags',
},
},
],
};
2 changes: 1 addition & 1 deletion src/components/inspectors/process.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-process',
},
items: [
{
Expand Down
7 changes: 4 additions & 3 deletions src/components/modeler/Modeler.vue
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,8 @@ import { id as laneId } from '../nodes/poolLane';
import { id as sequenceFlowId } from '../nodes/sequenceFlow';
import { id as associationId } from '../nodes/association';
import { id as messageFlowId } from '../nodes/messageFlow';
import { id as dataAssociationFlowId } from '../nodes/dataOutputAssociation';
import { id as dataOutputAssociationFlowId } from '../nodes/dataOutputAssociation/config';
import { id as dataInputAssociationFlowId } from '../nodes/dataInputAssociation/config';

import PaperManager from '../paperManager';
import registerInspectorExtension from '@/components/InspectorExtensionManager';
Expand Down Expand Up @@ -431,7 +432,7 @@ export default {

types.forEach(bpmnType => {
if (!this.parsers[bpmnType]) {
this.parsers[bpmnType] = { custom: [], implementation: [], default: [] };
this.parsers[bpmnType] = { custom: [], implementation: [], default: []};
}

if (customParser) {
Expand Down Expand Up @@ -746,7 +747,7 @@ export default {
store.commit('addNode', node);
this.poolTarget = null;

if ([sequenceFlowId, laneId, associationId, messageFlowId, dataAssociationFlowId].includes(node.type)) {
if ([sequenceFlowId, laneId, associationId, messageFlowId, dataOutputAssociationFlowId, dataInputAssociationFlowId].includes(node.type)) {
return;
}

Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/association/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-association',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/baseStartEvent/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-start-event',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/boundaryEvent/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-boundary-event',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/boundaryTimerEvent/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ export default merge(cloneDeep(boundaryEventConfig), {
config: {
label: 'Timing Control',
icon: 'clock',
name: 'inspector-accordion',
name: 'inspector-accordion-boundary-timer-event',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/dataInputAssociation/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-data-input-association',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/dataObject/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-data-object',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/dataOutputAssociation/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-data-output-association',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/dataStore/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-data-store',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/endEvent/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-end-event',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/eventBasedGateway/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-event-based-gateway',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/exclusiveGateway/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-exlcusive-gateway',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/gateway/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-gateway',
},
items: [
{
Expand Down
4 changes: 2 additions & 2 deletions src/components/nodes/inclusiveGateway/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-inclusive-gateway-config',
},
items: [
{
Expand All @@ -55,7 +55,7 @@ export default {
initiallyOpen: false,
label: 'Advanced',
icon: 'cogs',
name: 'inspector-accordion',
name: 'inspector-accordion-inclusive-gateway-advanced',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/intermediateEvent/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-intermediate-gateway-config',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/intermediateMessageEvent/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-intermediate-message-event',
},
items: [
{
Expand Down
4 changes: 2 additions & 2 deletions src/components/nodes/intermediateTimerEvent/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-intermediate-timer-config',
},
items: [
{
Expand All @@ -107,7 +107,7 @@ export default {
config: {
label: 'Timing Control',
icon: 'clock',
name: 'inspector-accordion',
name: 'inspector-accordion-intermediate-timer-event-timing-control',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/manualTask/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-manual-task',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/messageFlow/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-message-flow',
},
items: [
{
Expand Down
4 changes: 2 additions & 2 deletions src/components/nodes/parallelGateway/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-parallel-gateway-config',
},
items: [
{
Expand All @@ -52,7 +52,7 @@ export default {
initiallyOpen: false,
label: 'Advanced',
icon: 'cogs',
name: 'inspector-accordion',
name: 'inspector-accordion-parallel-gateway-direction',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/pool/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-pool',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/poolLane/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-pool-lane',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/scriptTask/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-script-task',
},
items: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/components/nodes/sequenceFlow/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ export default {
initiallyOpen: true,
label: 'Configuration',
icon: 'cog',
name: 'inspector-accordion',
name: 'inspector-accordion-sequence-flow',
},
items: [
{
Expand Down
Loading