12
0

Schematic Nodes #90

Zusammengeführt
Lixfel hat 48 Commits von schematic-node nach master 2021-11-20 13:12:32 +01:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
Lixfel hat 2021-01-31 11:18:54 +01:00 Änderungen angefragt
@ -32,16 +32,14 @@ import java.util.logging.Level;
public class CheckedSchematic {
Besitzer

CheckedSchematic kann ganz aus dem SpigotCore entfernt werden, ich glaube, kein System (im SpigotCore) braucht den noch.

CheckedSchematic kann ganz aus dem SpigotCore entfernt werden, ich glaube, kein System (im SpigotCore) braucht den noch.
Autor
Besitzer

Die SQL klasse ist aber nur Protected

Die SQL klasse ist aber nur Protected
Chaoscaot markierte diese Unterhaltung als gelöst
@ -34,3 +34,2 @@
private final String schemName;
private final int schemOwner;
private final int node;
Besitzer

Das rückwirkend für alle Schems (teils gelöschte) zu ändern? Halleluja! Viel Spaß.

Das rückwirkend für alle Schems (teils gelöschte) zu ändern? Halleluja! Viel Spaß.
Lixfel markierte diese Unterhaltung als gelöst
@ -33,1 +33,3 @@
public static String getLink(Schematic schem){
public static String getLink(SchematicNode schem){
if(schem.isDir())
throw new SecurityException("Can not Download Directorys");
Besitzer

Warum nicht? Wäre doch auch mal interessant. (Muss nicht im Rahmen dieses PRs umgesetzt werden)

Warum nicht? Wäre doch auch mal interessant. (Muss nicht im Rahmen dieses PRs umgesetzt werden)
Lixfel markierte diese Unterhaltung als gelöst
@ -35,35 +35,18 @@ import java.util.UUID;
public class Schematic {
Besitzer

Lass doch diese Klasse auch noch die Kernklasse sein. Es gibt keinen Grund, warum die Klasse jetzt plötzlich SchematicNode heißen muss. Provoziert viel weniger unnötige Änderungen, wenn das einfach gleichbleibt.

Lass doch diese Klasse auch noch die Kernklasse sein. Es gibt keinen Grund, warum die Klasse jetzt plötzlich SchematicNode heißen muss. Provoziert viel weniger unnötige Änderungen, wenn das einfach gleichbleibt.
Chaoscaot markierte diese Unterhaltung als gelöst
@ -16,3 +1,1 @@
You should have received a copy of the GNU Affero General Public License
along with this program. If not, see <https://www.gnu.org/licenses/>.
*/
/*
Besitzer

Unnötigste Änderung EU-West.

Unnötigste Änderung EU-West.
Autor
Besitzer

0Intellij Auto-Copyright, hatte die klasse gelöcht und neu erstellt :0

0Intellij Auto-Copyright, hatte die klasse gelöcht und neu erstellt :0
@ -26,3 +25,4 @@
import java.util.UUID;
import java.util.stream.Collectors;
public class SchematicMember {
Besitzer

Da das einfach (auch in anderen Systemen) nur unnötig viele Änderungen provoziert, obwohl sich häufig eigentlich nichts ändert: Behalte diesen alten Namen (und ggf. je nachdem, wie weit möglich) die alte API bei. Also NodeMember als SchematicMember lassen.

Da das einfach (auch in anderen Systemen) nur unnötig viele Änderungen provoziert, obwohl sich häufig eigentlich nichts ändert: Behalte diesen alten Namen (und ggf. je nachdem, wie weit möglich) die alte API bei. Also NodeMember als SchematicMember lassen.
Autor
Besitzer

????

????
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +33,4 @@
public class SchematicNode {
private static final String DIR_TYPE = "directory";
Besitzer

Wenns ein Verzeichnis ist: NULL

Wenns ein Verzeichnis ist: NULL
Chaoscaot markierte diese Unterhaltung als gelöst
Chaoscaot hat 2 Commits 2021-03-02 19:17:39 +01:00 hinzugefügt
Autor
Besitzer
INSERT INTO SchematicNode (NodeName, NodeOwner, NodeItem, NodeType, NodeRank, NodeFormat, NodeData, ParentNode) SELECT SchemName, SchemOwner, Item, SchemType, Rank, SchemFormat, SchemData, NULL FROM Schematic
``` INSERT INTO SchematicNode (NodeName, NodeOwner, NodeItem, NodeType, NodeRank, NodeFormat, NodeData, ParentNode) SELECT SchemName, SchemOwner, Item, SchemType, Rank, SchemFormat, SchemData, NULL FROM Schematic ```
Chaoscaot hat 1 Commit 2021-03-03 10:56:21 +01:00 hinzugefügt
Chaoscaot hat 1 Commit 2021-03-03 21:05:07 +01:00 hinzugefügt
Chaoscaot hat 1 Commit 2021-03-14 18:08:06 +01:00 hinzugefügt
Chaoscaot hat den Titel von WIP: Schematic Nodes zu Schematic Nodes 2021-03-14 18:43:14 +01:00 geändert
Chaoscaot hat ein Review von Lixfel 2021-03-25 09:06:58 +01:00 angefragt
Chaoscaot hat 1 Commit 2021-04-05 15:10:40 +02:00 hinzugefügt
Besitzer

Ok, da der Branch schon etwas auf der Halde liegt, gibt es die SQL-Befehle zum Konvertieren auf das neue System?

Hat dieses System Probleme, und wenn ja welche? Welche Lösungen sind "hacky" oder nichttrivial? Wie weit wurde der Branch getestet?

Damit das mal voran kommt :)

Ok, da der Branch schon etwas auf der Halde liegt, gibt es die SQL-Befehle zum Konvertieren auf das neue System? Hat dieses System Probleme, und wenn ja welche? Welche Lösungen sind "hacky" oder nichttrivial? Wie weit wurde der Branch getestet? Damit das mal voran kommt :)
Autor
Besitzer

Getestet auf die "alte" zugriffs Methode hatte ich getestet, sollte soweit im SchematicSystem/Fightsystem tun.

Das SQL-Statement zum konvertieren hatte ich mal hier drunter geschrieben:

INSERT INTO SchematicNode (NodeName, NodeOwner, NodeItem, NodeType, NodeRank, NodeFormat, NodeData, ParentNode) SELECT SchemName, SchemOwner, Item, SchemType, Rank, SchemFormat, SchemData, NULL FROM Schematic

Ansonsten sollte es eig. tun.

Getestet auf die "alte" zugriffs Methode hatte ich getestet, sollte soweit im SchematicSystem/Fightsystem tun. Das SQL-Statement zum konvertieren hatte ich mal hier drunter geschrieben: ``` INSERT INTO SchematicNode (NodeName, NodeOwner, NodeItem, NodeType, NodeRank, NodeFormat, NodeData, ParentNode) SELECT SchemName, SchemOwner, Item, SchemType, Rank, SchemFormat, SchemData, NULL FROM Schematic ``` Ansonsten sollte es eig. tun.
Chaoscaot hat 1 Commit 2021-05-17 09:26:25 +02:00 hinzugefügt
Besitzer

Membertransfer?

Membertransfer?
Autor
Besitzer
INSERT INTO NodeMember (NodeId, UserId) SELECT sn.NodeId, sm.Member FROM SchemMember AS sm LEFT JOIN SchematicNode as sn ON sm.SchemOwner = sn.NodeOwner AND sm.SchemName = sn.NodeName

sollte nach der Node gamcht werden

``` INSERT INTO NodeMember (NodeId, UserId) SELECT sn.NodeId, sm.Member FROM SchemMember AS sm LEFT JOIN SchematicNode as sn ON sm.SchemOwner = sn.NodeOwner AND sm.SchemName = sn.NodeName ``` sollte nach der Node gamcht werden
Chaoscaot hat den Titel von Schematic Nodes zu WIP: Schematic Nodes 2021-05-17 10:47:35 +02:00 geändert
Chaoscaot hat 3 Commits 2021-07-10 12:13:37 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2021-07-10 12:20:00 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2021-07-10 13:07:52 +02:00 hinzugefügt
Chaoscaot hat 2 Commits 2021-07-16 15:10:04 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2021-07-18 19:37:43 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2021-07-26 19:58:02 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2021-08-25 15:38:13 +02:00 hinzugefügt
Chaoscaot hat 2 Commits 2021-10-08 13:41:23 +02:00 hinzugefügt
# Conflicts:
#	SpigotCore_Main/src/de/steamwar/sql/Schematic.java
Updating SchematicNode
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
c1c04367e7
Chaoscaot hat 1 Commit 2021-10-09 22:18:31 +02:00 hinzugefügt
Merge remote-tracking branch 'origin/master' into schematic-node
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
87d43e608a
Chaoscaot hat 1 Commit 2021-10-11 11:25:15 +02:00 hinzugefügt
Fix schematic Loading
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
23136219dd
Chaoscaot hat 1 Commit 2021-11-03 16:13:06 +01:00 hinzugefügt
Merge remote-tracking branch 'origin/master' into schematic-node
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
1e3d387ff8
# Conflicts:
#	SpigotCore_Main/src/de/steamwar/sql/CheckedSchematic.java
#	SpigotCore_Main/src/de/steamwar/sql/DownloadSchematic.java
#	SpigotCore_Main/src/de/steamwar/sql/Schematic.java
Chaoscaot hat 1 Commit 2021-11-03 16:28:07 +01:00 hinzugefügt
Implement Streaming
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
de6d8bbb10
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-03 17:14:50 +01:00 hinzugefügt
Add Schematic SchematicNode Wrapper
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
37f11e4723
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Autor
Besitzer

Todo:

  • Schematic -> SchemNode Wrapper
  • Wrapper Test
  • "The Update" SQL-Statement
Todo: - [x] Schematic -> SchemNode Wrapper - [x] Wrapper Test - [ ] "The Update" SQL-Statement
Chaoscaot hat 1 Commit 2021-11-04 16:07:48 +01:00 hinzugefügt
Latest and Greatest Fixes!
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
1865342a00
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-05 16:31:19 +01:00 hinzugefügt
New Statement System
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
474212a340
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-05 22:09:27 +01:00 hinzugefügt
Safe Historic Data in CheckedNode
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
dfb38c7cda
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-05 22:25:02 +01:00 hinzugefügt
Refractor
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
0dd62f446b
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Lixfel hat 2021-11-06 14:29:02 +01:00 Änderungen angefragt
@ -0,0 +25,4 @@
import java.util.UUID;
public class CheckedNode {
Besitzer

Es bleibt bei CheckedSchematic - nur zukünftig mit zusätzlichem NodeID-Feld

Es bleibt bei CheckedSchematic - nur zukünftig mit zusätzlichem NodeID-Feld
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +59,4 @@
SchematicNode sNode = SchematicNode.getSchematicNode(node);
String nodeName = sNode.getName();
int nodeOwner = sNode.getOwner();
SQL.update("INSERT INTO CheckedSchematic" +
Besitzer

Statement-System

Statement-System
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +96,4 @@
}
public void remove() {
SQL.update("DELETE FROM CheckedSchematic WHERE NodeId", node);
Besitzer

.

.
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +24,4 @@
import java.util.HashSet;
import java.util.Set;
public class NodeMember {
Besitzer

Statement-System

Statement-System
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +412,4 @@
}
private void updateDB() {
SQL.update("UPDATE SchematicNode SET NodeName = ?, NodeOwner = ?, ParentNode = ?, NodeItem = ?, NodeType = ?, NodeRank = ? WHERE NodeId = ?",
Besitzer

Statement-System

Statement-System
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +422,4 @@
if (isDir()) {
getSchematicNodeInNode(getId()).forEach(SchematicNode::delete);
}
SQL.update("DELETE FROM SchematicNode WHERE NodeId = ?", id);
Besitzer

.

.
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +426,4 @@
}
public InputStream schemData() throws IOException {
ResultSet rs = SQL.select("SELECT NodeData FROM SchematicNode WHERE NodeId = ?", id);
Besitzer

.

.
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +469,4 @@
}
private void updateDatabase(InputStream blob, boolean newFormat) {
SQL.update("UPDATE SchematicNode SET NodeData = ?, NodeFormat = ? WHERE NodeId = ?", blob, newFormat, id);
Besitzer

.

.
Chaoscaot markierte diese Unterhaltung als gelöst
Chaoscaot hat 1 Commit 2021-11-06 14:42:39 +01:00 hinzugefügt
Fixes 'n' Statements
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
1d9532d971
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-06 14:44:29 +01:00 hinzugefügt
Fix Copyright
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
3a06ecedf1
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 2021-11-11 21:35:15 +01:00 das Fälligkeitsdatum 2021-11-12 hinzugefügt
Chaoscaot hat eine neue Abhängigkeit 2021-11-11 21:42:37 +01:00 hinzugefügt
Chaoscaot hat 1 Commit 2021-11-11 21:42:51 +01:00 hinzugefügt
Merge branch 'master' into schematic-node
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
32a9e2f619
Chaoscaot hat eine Abhängigkeit 2021-11-11 21:42:58 +01:00 entfernt
Chaoscaot hat eine neue Abhängigkeit 2021-11-11 21:43:38 +01:00 hinzugefügt
Chaoscaot ändert das Abgabedatum von 2021-11-12 auf 2021-11-13 2021-11-11 22:03:03 +01:00 s
Chaoscaot hat eine Abhängigkeit 2021-11-12 15:53:25 +01:00 entfernt
Chaoscaot hat 1 Commit 2021-11-13 15:10:03 +01:00 hinzugefügt
Some Crazy-Ass SQL-Statements and Performance!
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
c49daba060
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-13 16:13:58 +01:00 hinzugefügt
Fixing Method renaming
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
48b2733470
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-14 11:27:22 +01:00 hinzugefügt
SchematicNode Fixes
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
ee06dc74c7
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-14 11:32:14 +01:00 hinzugefügt
Fix
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
20b8dd568f
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat den Titel von WIP: Schematic Nodes zu Schematic Nodes 2021-11-14 11:40:44 +01:00 geändert
Chaoscaot hat 1 Commit 2021-11-14 12:00:48 +01:00 hinzugefügt
Yes
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
794791145b
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat ein Review von Lixfel 2021-11-14 12:00:59 +01:00 angefragt
Chaoscaot hat 1 Commit 2021-11-15 23:50:02 +01:00 hinzugefügt
New Function
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
2d87feb634
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-20 11:26:35 +01:00 hinzugefügt
Fixing Statements
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
da2dfc7a33
Signed-off-by: Chaoscaot <chaoscaot444@gmail.com>
Chaoscaot hat 1 Commit 2021-11-20 11:59:55 +01:00 hinzugefügt
Merge branch 'master' into schematic-node
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
136400f861
Lixfel hat die Änderungen 2021-11-20 12:46:44 +01:00 genehmigt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Warten auf Ende der Datenbankänderungen.

Warten auf Ende der Datenbankänderungen.
Lixfel hat Commit 8095819691 in master 2021-11-20 13:12:32 +01:00 gemerged
Lixfel löschte die Branch schematic-node 2021-11-20 13:12:35 +01:00
Anmelden, um an der Diskussion teilzunehmen.
Keine Beschreibung angegeben.