OneForAll - Set image to inactive won't save

[CodeALot]

OneForAll 2.0.2.23

• Go into modify_item.php  and set one or more images to "Active: NO"
• Save
• Go back into the item.
• Images are all set to "Active: YES" 

• Go into the database.
• Find table mod_oneforall_image s
• Set an image to "Active: 0"
• Go back into modify_item.php of the item that you just set an image inactive for
• You'll see that the image you set to inactive in the database, is now indeed inactive.
• Click the checkbox to make it Active again.
• Save the item.
• Go back
• The image is now Active again.

So, I can't set an image to INactive, but I CAN set it to Active.
 
What's wrong here? It did work as expected in earlier versions.

(Sorry, this should have gone into a different forum board: WebsiteBaker 2.12.2 --> Modules)

[jacobi22]

works for me

the only point, what i see:  go into save_item.php  ~ Line 491

and add here the integer-specification  (int) to the $image[active] like this

Code: [Select]

    // Update image data
    $database->query('UPDATE `'.TABLE_PREFIX.'mod_'.$mod_name.'_images` '
                    .'SET   `active` = '.(int)$image['active'].', '
                    .'         `alt` = \''.$image['alt'].'\', '
                    .'       `title` = \''.$image['title'].'\', '
                    .'     `caption` = \''.$image['caption'].'\' '
                    .'WHERE `img_id` = '.(int)$img_id.' ');

[dbs]

With or without (int) it works for me also not.

[CodeALot]

Unfortunately, that did not do the trick for me
Thanks for your reply, though

[CodeALot]

With or without (int) it works for me also not.

You mean you have the same problem?

[dbs]

Yes.
Try this in line 489:

Code: [Select]

                    .'SET   `active` = \''.$image['active'].'\', '
I think the field type in db is enum and expects a string.

[CodeALot]

Yes.
Try this in line 489:

Code: [Select]

                    .'SET   `active` = \''.$image['active'].'\', '
I think the field type in db is enum and expects a string.

In the table definition it says:

   active   enum('1', '0')   utf8_general_ci   

But your solution worked! Thanks a lot!

[jacobi22]

i think, the "secret" and the different is the MYSQL-Version - but i'm not sure, that i understand everything correct here -> https://dev.mysql.com/doc/refman/8.0/en/enum.html

i understand: if i use a unmasked number, it works as an index from the enum-field -

submitted 1 == index(1) = in this case: 1
submitted 2 == index(2) = in this case: 0

and a masked value (like  \''.$image['active'].'\') is the value like 1 or 0

may and hope, Manu or Dietmar can help, because OFA and Bakery are full of enum-fields like this

[CodeALot]

The weird thing is that it DID work if you had an INactive image and you set it to Active, it would save correctly.
Only setting it back to INactive was not saved...

[jacobi22]

if you activate the checkbox for image-activity, you submit a "1", but if you not activate this checkbox, you submit nothing (for this activity-setting) - thats the result, if you use 1 or 0 in a checkbox (also in a radio-button)
(If you use instead of 1 or 0 - now 1 or 2, you submit a result in every case, but it needs one or two lines more in the code of save_image.php)

next part is the save_image - the same procedure....
if the field "image_active" == empty (means: nothing submitted), set the Value to 0
if it's not empty (means: something submitted, no matter, what), set it to 1

now, we have a numeric value like 1 or 0. in older mysql-version's it doesn't matter, what kind of type you have, integer or string, maybe important, if you use MYSQL-STRICT, i've not test it.
but in the newer version, you can submitted this value as string (like post from dbs) or as numeric index

for example: you use a enum-field like enum('Berlin','Paris','London')
a submitted 1 as integer is here == Berlin, because 'Berlin" is the first definition (or Index) in the field definition), a submitted 3 is == London, a submitted 4 == nothing, because, you have only 3 indizes, not 4 - #4 is invalid). if you have "nothing", use the default value or (if not defined) do nothing

back to OFA
definition of this database field: enum('1','0') default '1'
Index 1 == 1
Index 2 == 0

but you submit not valid index, you submit index == 0. And if you submit a invalide index, it use the default value == 1

Solution from dbs submitted a string ( not the number of the index like my first solution), that's why it works also with 0

[CodeALot]

Very clear explanation, thanks!