Skip to content

Commit d17e4a1

Browse files
authored
Merge branch '4.1-develop' into 4.1.17
2 parents e2b6753 + 7487afd commit d17e4a1

File tree

11 files changed

+52
-15
lines changed

11 files changed

+52
-15
lines changed

ProcessMaker/Http/Controllers/Api/ProcessRequestController.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,6 +136,10 @@ public function index(Request $request)
136136
}
137137
}
138138

139+
if (! $user->can('view-all_requests')) {
140+
$query->pmql('requester = "' . $user->username . '" OR participant = "' . $user->username . '"');
141+
}
142+
139143
$query->nonSystem();
140144

141145
try {
@@ -375,7 +379,7 @@ public function activateIntermediateEvent(ProcessRequest $request, $event)
375379
{
376380
// Get the process definition
377381
$process = $request->process;
378-
$catchEvent = $process->getDefinitions()->findElementById($event)->getBpmnElementInstance();
382+
$catchEvent = $request->getVersionDefinitions()->findElementById($event)->getBpmnElementInstance();
379383
if (!($catchEvent instanceof CatchEventInterface)) {
380384
return abort(423, __('Invalid element, not a catch event ' . get_class($catchEvent)));
381385
}

ProcessMaker/Managers/TaskSchedulerManager.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,9 +259,10 @@ public function executeIntermediateTimerEvent(ScheduledTask $task, $config)
259259
}
260260

261261
$process = Process::find($id);
262+
/** @var ProcessRequest $request */
262263
$request = ProcessRequest::find($task->process_request_id);
263264

264-
$definitions = $process->getDefinitions();
265+
$definitions = $request->getVersionDefinitions();
265266
if (!$definitions->findElementById($config->element_id)) {
266267
return;
267268
}

ProcessMaker/Models/Process.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -490,7 +490,7 @@ public function getNextUser(ActivityInterface $activity, ProcessRequestToken $to
490490
return $userByRule;
491491
}
492492

493-
$definitions = $token->getInstance()->process->getDefinitions();
493+
$definitions = $token->getInstance()->getVersionDefinitions();
494494
$properties = $definitions->findElementById($activity->getId())->getBpmnElementInstance()->getProperties();
495495
$assignmentLock = array_key_exists('assignmentLock', $properties) ? $properties['assignmentLock'] : false;
496496

ProcessMaker/Models/ProcessRequest.php

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
use Spatie\MediaLibrary\HasMedia\HasMedia;
2323
use Spatie\MediaLibrary\HasMedia\HasMediaTrait;
2424
use Throwable;
25-
use Illuminate\Database\Eloquent\ModelNotFoundException;
25+
use ProcessMaker\Repositories\BpmnDocument;
2626
use ProcessMaker\Traits\HideSystemResources;
2727

2828
/**
@@ -42,6 +42,7 @@
4242
* @property \Carbon\Carbon $created_at
4343
* @property Process $process
4444
* @property ProcessRequestLock[] $locks
45+
* @method static ProcessRequest find($id)
4546
*
4647
* @OA\Schema(
4748
* schema="processRequestEditable",
@@ -774,4 +775,18 @@ public function hasLock(ProcessRequestLock $lock)
774775
$first = $this->locks()->orderBy('id')->first();
775776
return !$first || $first->getKey() === $lock->getKey();
776777
}
778+
779+
/**
780+
* Get the BPMN definitions version of the process that is running.
781+
*
782+
* @param boolean $forceParse
783+
* @param mixed $engine
784+
*
785+
* @return BpmnDocument
786+
*/
787+
public function getVersionDefinitions($forceParse = false, $engine = null)
788+
{
789+
$processVersion = $this->processVersion ?: $this->process;
790+
return $processVersion->getDefinitions($forceParse, $engine);
791+
}
777792
}

ProcessMaker/Models/ProcessRequestToken.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,9 @@ public function getDefinition($asObject = false, $par = null)
301301
*/
302302
public function getBpmnDefinition()
303303
{
304-
$definitions = $this->processRequest->process->getDefinitions();
304+
/** @var ProcessRequest $request */
305+
$request = $this->processRequest ?: $this->getInstance();
306+
$definitions = $request->getVersionDefinitions();
305307
return $definitions->findElementById($this->element_id);
306308
}
307309

database/migrations/2020_11_16_215424_create_security_logs_table.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
use Illuminate\Database\Schema\Blueprint;
55
use Illuminate\Support\Facades\Schema;
66
use ProcessMaker\Models\Permission;
7+
use ProcessMaker\Models\SecurityLog;
78

89
class CreateSecurityLogsTable extends Migration
910
{
@@ -14,15 +15,14 @@ class CreateSecurityLogsTable extends Migration
1415
*/
1516
public function up()
1617
{
17-
Schema::create('security_logs', function (Blueprint $table) {
18+
$model = new SecurityLog;
19+
Schema::connection($model->getConnectionName())->create('security_logs', function (Blueprint $table) {
1820
$table->bigIncrements('id');
1921
$table->string('event', 40)->index();
2022
$table->string('ip', 40)->index()->nullable();
2123
$table->json('meta')->nullable();
2224
$table->unsignedInteger('user_id')->nullable();
2325
$table->timestamp('occurred_at');
24-
25-
$table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
2626
});
2727

2828
if (! Permission::where('name', 'view-security-logs')->first()) {

package-lock.json

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464
"@processmaker/modeler": "^1.6.1",
6565
"@processmaker/processmaker-bpmn-moddle": "^0.11.0",
6666
"@processmaker/screen-builder": "^2.5.2",
67-
"@processmaker/vue-form-elements": "^0.18.4",
67+
"@processmaker/vue-form-elements": "^0.19.1",
6868
"babel-eslint": "^10.1.0",
6969
"babel-plugin-transform-object-rest-spread": "^6.26.0",
7070
"bootstrap": "^4.5.3",

resources/js/data/datetime_formats.json

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,12 @@
2323
"calendarFormat": "dd/LL/yyyy",
2424
"title": "d/m/Y (31/12/2017)"
2525
},
26+
{
27+
"format": "d/m/Y h:i A",
28+
"momentFormat": "DD/MM/YYYY h:mm A",
29+
"calendarFormat": "dd/LL/yyyy h:mm a",
30+
"title": "d/m/Y h:i A (31/12/2017 11:30 pm)"
31+
},
2632
{
2733
"format": "d/m/Y H:i",
2834
"momentFormat": "DD/MM/YYYY HH:mm",

resources/sass/app.scss

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -456,9 +456,11 @@ h1.page-title {
456456
padding: 0 10px;
457457
}
458458

459-
// Data table rules
460-
459+
.b-tooltip:not([style*='transform']) {
460+
top: 0;
461+
}
461462

463+
// Data table rules
462464

463465
[class^='icon-'],
464466
[class*=' icon-'] {

0 commit comments

Comments
 (0)