#121 Plugin "SensitiveContent" messes up proportion in attachments

I had it activated since... i don't remember. Attachments in the two nodes i administer were showing with no proportion; through setconfig.php i managed to show them a bit better. Disabling the plugin brings everything back to normal.

Wait, all attachments were affected by this plugin?

Yes, both nodes showed that behaviour on all attachments.

@aab my latest commit should fix this issue, but I don't really have a way to test it

I've just tried on one of the nodes; as soon as the plugin is activated, proportion is gone.

What do you mean by proportion? I might be misunderstanding you

Ok, i'll try to explain better with images.

Now, the one without is specially distorted, as i I had to change width and height thumbnails with setconfig in order to "fix" it: most of thumbnails fit in 600x90, as they are links, and not images like this. (Edit)Ah, and it doesn't depend on the theme used.

Fixed with 9b587ae4a3

as seen in https://social.hackersatporto.com/conversation/5622#notice-11624

Ok, sorry to reopen this, just a couple of things:

• In the new README, this is the example:

addPlugin('SensitiveContent', ['hideforvisitors' => true]

But it does not work unless i write

addPlugin('SensitiveContent', ['hideforvisitors' => true]);

• Appart from that, i keep having distorted images with the plugin. I know it has been fixed, and i know why they keep being distorted: before i knew this plugin was responsible, i used setconfig.php to give thumbnails a fixed measure of 600x90. I've tried restoring them to the default value, 450x600:

https://khp.ignorelist.com/conversation/206812#notice-426573

Is there any way to set this to "auto" (or something like that), so the thumbnails get the proportion they need?

Fixed the README.

run ./scripts/clean_thumbnails.php

Relevant settings concerning thumbs: https://notabug.org/diogo/gnu-social/src/nightly/lib/util/default.php#L285-L289

• crop: Crop to the given dimensions (not preserving aspect ratio)
• maxsize: width and height won't be larger than this value (just re-read the code and, I don't think this setting was needed at all, it would be more helpful if it was an actual max size for a thumb file or smth...)
• width: Max width
• height: Max height
• upscale: Whether to allow a thumb to acquire greater dimensions than its original file or not

It seems i got it just like the default values:

thumbnail            dir                  '/var/www/html/file/thumb'
thumbnail            path                 NULL
thumbnail            server               NULL
thumbnail            crop                 'false'
thumbnail            maxsize              1000
thumbnail            width                '450'
thumbnail            height               '600'
thumbnail            upscale              'false'
thumbnail            animated             'false'

The only difference i can see is that "dir" is defined.

I've executed clean_thumbnails, but there has been no change.

Just in case, i've looked in db, everything seems fine...

MariaDB [gnusocial]> select * from config where section = 'thumbnail';
+-----------+----------+-------+
| section   | setting  | value |
+-----------+----------+-------+
| thumbnail | animated | false |
| thumbnail | crop     | false |
| thumbnail | height   | 600   |
| thumbnail | upscale  | false |
| thumbnail | width    | 450   |
+-----------+----------+-------+

I've checked the plugin today after all this time, i'm afraid there's still disproportion in images: https://gnusocial.net/conversation/4963984#notice-8172421

setconfig.php -a | grep thumbnail gives

thumbnail            dir                  '/var/www/gnusocial/file/thumb'
thumbnail            path                 NULL
thumbnail            server               NULL
thumbnail            crop                 'false'
thumbnail            maxsize              1000
thumbnail            width                450
thumbnail            height               '600'
thumbnail            upscale              'false'
thumbnail            animated             'false'
thumbnail            file_quota           '2097152'
thumbnail            process_links        'true'
thumbnail            show_thumbs          'true'

MariaDB [gnusocial]> select * from config where section = 'thumbnail';
+-----------+---------------+---------+
| section   | setting       | value   |
+-----------+---------------+---------+
| thumbnail | animated      | false   |
| thumbnail | crop          | false   |
| thumbnail | file_quota    | 2097152 |
| thumbnail | height        | 600     |
| thumbnail | process_links | true    |
| thumbnail | show_thumbs   | true    |
| thumbnail | upscale       | false   |
+-----------+---------------+---------+

I've also run clean_thumbnails. Anything else i'm missing?

Fixed with "php scripts/setconfig.php thumbnail crop 0"