Kan ik deze queries combineren als 1

misschien met Union
query1 Union query2 Union query 3 .....

Nee Union is niet goed, iig bedankt

[size=xsmall]Toevoeging op 21/12/2017 16:35:29:[/size]

Jan excuus voor mijn vorige reactie, ik heb me wat verder verdiept in UNION en heb de query nu zo:

SELECT
    'att' as type, 
    al.attribute_id, al.attribute_label label,
    la.lightspeed_id
FROM
    lightspeed_types lt
JOIN
    attributes_categories ac
    ON ac.category_id = lt.category_id

JOIN
    attributes_lang al
    ON al.attribute_id = ac.attribute_id
    AND al.lang_id = 'NL'
LEFT JOIN
    lightspeed_attributes la
    ON la.attribute_id = ac.attribute_id
WHERE
    la.lightspeed_id IS NULL
UNION
SELECT
    'spec',
    sl.spec_id, sl.specification_label label,
    la.lightspeed_id
FROM
    lightspeed_types lt
JOIN
    specifications_categories sc
    ON sc.category_id = lt.category_id

JOIN
    specifications_lang sl
    ON sl.spec_id = sc.spec_id
    AND sl.lang_id = 'NL'
LEFT JOIN
    lightspeed_attributes la
    ON la.spec_id = sc.spec_id
WHERE
    la.lightspeed_id IS NULL

Technisch wellicht mogelijk (immers het werkt), maar misschien wel lastig(er) te begrijpen en debuggen in de toekomst...

Yup, en je wint er niets mee. Je maakt je code alleen maar omslachtiger.

Die UNION ziet er eigenaardig uit. De onderdelen van een UNION moeten precies dezelfde kolommen teruggeven, anders zal het niet werken.

Zou je eens kunnen beschrijven wat voor recordset je precies uit je database wilt halen en wat de structuur is van de tabellen die hierbij betrokken zijn?

Ben, bedoel je dat ik beter de 2 afzonderlijke queries had kunnen gebruiken.

Jan morgen zal ik dat bewerkstelligen, alvast bedankt voor alle moeite




[size=xsmall]Toevoeging op 22/12/2017 08:35:19:[/size]

mysql> describe lightspeed_types;
+---------------+-----------+------+-----+-------------------+-----------------------------+
| Field         | Type      | Null | Key | Default           | Extra                       |
+---------------+-----------+------+-----+-------------------+-----------------------------+
| category_id   | int(9)    | NO   | MUL | NULL              |                             |
| lightspeed_id | int(11)   | NO   | PRI | NULL              |                             |
| updated_at    | timestamp | NO   |     | CURRENT_TIMESTAMP | on update CURRENT_TIMESTAMP |
+---------------+-----------+------+-----+-------------------+-----------------------------+

[size=xsmall]Toevoeging op 22/12/2017 08:39:32:[/size]

mysql> describe attributes_categories;
+--------------+---------------------+------+-----+---------+-------+
| Field        | Type                | Null | Key | Default | Extra |
+--------------+---------------------+------+-----+---------+-------+
| attribute_id | bigint(20) unsigned | NO   | MUL | NULL    |       |
| category_id  | int(11) unsigned    | NO   | MUL | NULL    |       |
| sort_order   | int(11)             | NO   | MUL | 0       |       |
+--------------+---------------------+------+-----+---------+-------+

mysql> describe attributes_lang;
+-----------------+---------------------+------+-----+---------+----------------+
| Field           | Type                | Null | Key | Default | Extra          |
+-----------------+---------------------+------+-----+---------+----------------+
| attribute_id    | bigint(20) unsigned | NO   | PRI | NULL    | auto_increment |
| lang_id         | varchar(2)          | NO   | PRI |         |                |
| attribute_label | varchar(250)        | NO   |     |         |                |
+-----------------+---------------------+------+-----+---------+----------------+

mysql> describe lightspeed_attributes;
+-------------------+-----------+------+-----+-------------------+-----------------------------+
| Field             | Type      | Null | Key | Default           | Extra                       |
+-------------------+-----------+------+-----+-------------------+-----------------------------+
| attribute_id      | int(9)    | YES  |     | NULL              |                             |
| spec_id           | int(9)    | YES  |     | NULL              |                             |
| lightspeed_id     | int(11)   | NO   | PRI | NULL              |                             |
| is_type_attribute | int(1)    | NO   |     | 0                 |                             |
| updated_at        | timestamp | NO   |     | CURRENT_TIMESTAMP | on update CURRENT_TIMESTAMP |
+-------------------+-----------+------+-----+-------------------+-----------------------------+

[size=xsmall]Toevoeging op 22/12/2017 08:41:18:[/size]

mysql> describe specifications_categories;
+-------------+---------------------+------+-----+---------+-------+
| Field       | Type                | Null | Key | Default | Extra |
+-------------+---------------------+------+-----+---------+-------+
| spec_id     | bigint(20) unsigned | NO   | MUL | NULL    |       |
| category_id | int(11) unsigned    | NO   | MUL | NULL    |       |
| sort_order  | int(11)             | NO   | MUL | 0       |       |
+-------------+---------------------+------+-----+---------+-------+

mysql> describe specifications_lang;
+---------------------+---------------------+------+-----+---------+----------------+
| Field               | Type                | Null | Key | Default | Extra          |
+---------------------+---------------------+------+-----+---------+----------------+
| spec_id             | bigint(20) unsigned | NO   | PRI | NULL    | auto_increment |
| lang_id             | varchar(2)          | NO   | PRI |         |                |
| specification_label | varchar(250)        | NO   |     |         |                |
+---------------------+---------------------+------+-----+---------+----------------+

[size=xsmall]Toevoeging op 22/12/2017 08:43:20:[/size]

Dit is het resultaat van de UNION query:

+------+--------------+--------------------+---------------+
| type | attribute_id | label              | lightspeed_id |
+------+--------------+--------------------+---------------+
| att  |            1 | Opslagcapaciteit   |          NULL |
| att  |            2 | Productsoort       |          NULL |
| att  |            3 | Voor wie           |          NULL |
| att  |            4 | Flacon             |          NULL |
| att  |            5 | Test Geuren        |          NULL |
| att  |            7 | Test 2             |          NULL |
| spec |            1 | geluidsniveau (db) |          NULL |
| spec |            2 | vermogen (watt)    |          NULL |
| spec |            3 | Type               |          NULL |
| spec |            4 | Geurnoot           |          NULL |
+------+--------------+--------------------+---------------+

[size=xsmall]Toevoeging op 22/12/2017 08:44:54:[/size]

En zo schrijf ik het nu weg:

function createAttributes($lang){
        $attributes = $this->m_lightspeed->getAttributes(TRUE, strtoupper($lang));
        dump($attributes);
        foreach ($attributes as $value):
            
            $attribute = $this->api()->attributes->create([
                "title" => $value->label
            ]);
        
            if($value->type == 'att'):
                $this->m_lightspeed->saveAttributes($value->attribute_id, NULL, $attribute['id']);
            else:
                $this->m_lightspeed->saveAttributes(NULL, $value->attribute_id, $attribute['id']);    
            endif;

        endforeach;
        
        redirect('dashboard');
    }

Jan Koehoorn op 21/12/2017 22:24:59

Die UNION ziet er eigenaardig uit. De onderdelen van een UNION moeten precies dezelfde kolommen teruggeven, anders zal het niet werken.

Deze opmerking is onjuist, doorgaans gaat het goed als het aantal kolommen per select maar gelijk is zodat de union klopt. De inhoud of kolomnamen hoeven niet hetzelfde te zijn, ook de tabel kan per per select in de union een andere zijn. Of dat zinvol is hangt af van de informatiebehoefte en datamodellering uiteraard. Daarnaast kan er wel een issue zijn met het datatype. Maar het kan werken!