Differential D4376

Avoid a db error if the folder already exists and return status 2
ClosedPublic
Actions

Authored by mollekopf on Jun 15 2023, 1:38 PM.

Details

Reviewers

machniak

Group Reviewers

Restricted Project

Commits

rSfb06ebbc6975: Avoid a db error if the folder already exists and return status 2

Summary

Also, be a bit more verbose on errors

Diff Detail

Repository

rS syncroton

Branch

master

Lint

Lint Skipped

Unit

No Test Coverage

Build Status

Buildable 43379
Build 17339: arc lint + arc unit

Event Timeline

mollekopf requested review of this revision.Jun 15 2023, 1:38 PM

mollekopf created this revision.

Harbormaster completed remote builds in B43370: Diff 12548.Jun 15 2023, 1:38 PM

mollekopf added a reviewer: Restricted Project.Jun 15 2023, 1:38 PM

take 2

Harbormaster completed remote builds in B43376: Diff 12554.Jun 15 2023, 1:51 PM

warn on all exceptions

Harbormaster completed remote builds in B43379: Diff 12560.Jun 15 2023, 1:59 PM

machniak accepted this revision.Jun 16 2023, 8:02 AM

This revision is now accepted and ready to land.Jun 16 2023, 8:02 AM

Closed by commit rSfb06ebbc6975: Avoid a db error if the folder already exists and return status 2. · Explain WhyJun 16 2023, 12:59 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

lib/

ext/

Syncroton/

Command/

FolderCreate.php

21 lines

Diff 12560

View Options

lib/ext/Syncroton/Command/FolderCreate.php

Avoid a db error if the folder already exists and return status 2ClosedPublicActions