From d5233d83d90995ffc3611af25f56ae9534d5850f Mon Sep 17 00:00:00 2001
From: Patrick Schratz <patrick.schratz@gmail.com>
Date: Fri, 10 Jan 2025 10:23:16 +0100
Subject: [PATCH 1/2] Reduce interactive WP icon size (#4690)

---
 web/src/components/repo/pipeline/PipelineItem.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/src/components/repo/pipeline/PipelineItem.vue b/web/src/components/repo/pipeline/PipelineItem.vue
index 663a91db2..c17b6d0e5 100644
--- a/web/src/components/repo/pipeline/PipelineItem.vue
+++ b/web/src/components/repo/pipeline/PipelineItem.vue
@@ -11,7 +11,7 @@
           'bg-wp-state-info-100': pipelineStatusColors[pipeline.status] === 'blue',
         }"
       />
-      <div class="flex h-full w-8 flex-wrap items-center justify-between">
+      <div class="flex h-full w-6 flex-wrap items-center justify-between">
         <PipelineRunningIcon v-if="pipeline.status === 'started' || pipeline.status === 'running'" />
         <PipelineStatusIcon v-else class="mx-2 md:mx-3" :status="pipeline.status" />
       </div>

From 628c0e8faa1c1d4299e053b666500c2dbb383e37 Mon Sep 17 00:00:00 2001
From: Patrick Schratz <patrick.schratz@gmail.com>
Date: Fri, 10 Jan 2025 18:12:21 +0100
Subject: [PATCH 2/2] Don't fail user creation if org exists (#4687)

---
 server/store/datastore/org.go        | 10 +++++--
 server/store/datastore/user.go       | 21 +++++++++++++--
 server/store/datastore/users_test.go | 39 ++++++++++++++++++++++++++++
 3 files changed, 66 insertions(+), 4 deletions(-)

diff --git a/server/store/datastore/org.go b/server/store/datastore/org.go
index eb18ac8df..59fa18f80 100644
--- a/server/store/datastore/org.go
+++ b/server/store/datastore/org.go
@@ -77,9 +77,15 @@ func (s storage) orgDelete(sess *xorm.Session, id int64) error {
 func (s storage) OrgFindByName(name string) (*model.Org, error) {
 	// sanitize
 	name = strings.ToLower(name)
-	// find
 	org := new(model.Org)
-	return org, wrapGet(s.engine.Where("name = ?", name).Get(org))
+	has, err := s.engine.Where("name = ?", name).Get(org)
+	if err != nil {
+		return nil, fmt.Errorf("failed to check if org exists: %w", err)
+	}
+	if !has {
+		return nil, nil
+	}
+	return org, nil
 }
 
 func (s storage) OrgRepoList(org *model.Org, p *model.ListOptions) ([]*model.Repo, error) {
diff --git a/server/store/datastore/user.go b/server/store/datastore/user.go
index aa3d35f7d..deeddf542 100644
--- a/server/store/datastore/user.go
+++ b/server/store/datastore/user.go
@@ -15,6 +15,8 @@
 package datastore
 
 import (
+	"fmt"
+
 	"xorm.io/xorm"
 
 	"go.woodpecker-ci.org/woodpecker/v3/server/model"
@@ -59,9 +61,24 @@ func (s storage) CreateUser(user *model.User) error {
 		Name:   user.Login,
 		IsUser: true,
 	}
-	err := s.orgCreate(org, sess)
+
+	existingOrg, err := s.OrgFindByName(org.Name)
 	if err != nil {
-		return err
+		return fmt.Errorf("failed to check if org exists: %w", err)
+	}
+
+	if existingOrg != nil {
+		if existingOrg.Name == user.Login {
+			err = s.OrgUpdate(org)
+			if err != nil {
+				return fmt.Errorf("failed to update existing org: %w", err)
+			}
+		}
+	} else {
+		err = s.orgCreate(org, sess)
+		if err != nil {
+			return fmt.Errorf("failed to create new org: %w", err)
+		}
 	}
 	user.OrgID = org.ID
 	// only Insert set auto created ID back to object
diff --git a/server/store/datastore/users_test.go b/server/store/datastore/users_test.go
index 0189e9f49..410daa36f 100644
--- a/server/store/datastore/users_test.go
+++ b/server/store/datastore/users_test.go
@@ -94,3 +94,42 @@ func TestUsers(t *testing.T) {
 	_, err3 := store.GetUser(getUser.ID)
 	assert.Error(t, err3)
 }
+
+func TestCreateUserWithExistingOrg(t *testing.T) {
+	store, closer := newTestStore(t, new(model.User), new(model.Org), new(model.Perm))
+	defer closer()
+
+	existingOrg := &model.Org{
+		ForgeID: 1,
+		IsUser:  true,
+		Name:    "existingorg",
+		Private: false,
+	}
+
+	err := store.OrgCreate(existingOrg)
+	assert.NoError(t, err)
+	assert.EqualValues(t, "existingorg", existingOrg.Name)
+
+	// Create a new user with the same name as the existing organization
+	newUser := &model.User{
+		Login: "existingOrg",
+		Hash:  "A",
+	}
+	err = store.CreateUser(newUser)
+	assert.NoError(t, err)
+
+	updatedOrg, err := store.OrgGet(existingOrg.ID)
+	assert.NoError(t, err)
+	assert.Equal(t, "existingorg", updatedOrg.Name)
+
+	newUser2 := &model.User{
+		Login: "new-user",
+		Hash:  "B",
+	}
+	err = store.CreateUser(newUser2)
+	assert.NoError(t, err)
+
+	newOrg, err := store.OrgFindByName("new-user")
+	assert.NoError(t, err)
+	assert.Equal(t, "new-user", newOrg.Name)
+}