Check for uniq records

[T0mWz]

Ensure that records are uniq for user-group relation;

MariaDB [miskatonic_tst_oc]> select * from oc_fg_groups;
+----------------------------------------------------+
| gid                                                |
+----------------------------------------------------+
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) |
+----------------------------------------------------+
1 row in set (0.001 sec)

MariaDB [miskatonic_tst_oc]> select * from oc_fg_group_user;
+----------------------------------------------------+------------------+
| gid                                                | uid              |
+----------------------------------------------------+------------------+
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | wezep001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | zoete001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | wezep001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | wezep001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | zoete001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | zoete001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | wezep001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | zoete001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | zoete001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | wezep001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | zoete001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | wezep001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | wezep001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | zoete001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | wezep001@surf.nl |
| TestGroup (uniharderwijk_surfdrive_test) (SRAM CO) | zoete001@surf.nl |
+----------------------------------------------------+------------------+
16 rows in set (0.003 sec)

[navid-shokri]

@soltanireza65 Can you check this Issue?

[soltanireza65]

@shokri-navid Yes i will

[soltanireza65]

I think the best way to handle this is to update the migration file to make 2 columns qunit together, instead of checking it in the backend, (also this is the current way that we are using and its somehow not working)

[T0mWz]

Maybe also nicer to user the iSchemaMigration option instead of the current migration. Where we not using MyISAM for example and with the iSchemaMigration you fix it for all type of DBs; Example of an iSchemaMigration can you find here; https://github.com/owncloud/oauth2/blob/master/appinfo/Migrations/Version20161122085340.php#L7

[soltanireza65]

Maybe also nicer to user the iSchemaMigration option instead of the current migration. Where we not using MyISAM for example and with the iSchemaMigration you fix it for all type of DBs; Example of an iSchemaMigration can you find here; owncloud/oauth2@master/appinfo/Migrations/Version20161122085340.php#L7

ooo it uses XML marckups as SQL schemas 😃😃😃 I have never seen it before, great...