3UU

If do you follow the CVE links you will end up https://plugins.trac.www.ads-software.com/changeset?sfp_email=&sfph_mail=&reponame=&new=3069111%40shariff%2Ftrunk&old=3061040%40shariff%2Ftrunk&sfp_email=&sfph_mail=#file1 As you can see with the code changes version 4.6.13 has addressed this issue. However I would recommand to use the recent 6.6.14 because there is an other fix (more importand!!!) to the share count api. So please update. BTW: Please also open the admin page within WP because this will do some “self healing” to to option if needed. Update to 4.6.14, open the admin page, save it and you are done ??

Ich kann es aktuell leider nicht testen. Aber koenntest Du mal in der wp-content/plugins/shariff/shariff.php an Zeile 195 ein

‘permission_callback’ => ‘__return_true’,

einfuegen. Der Code sollte dann inentwa so aussehen

                array(
                        'methods'  => 'GET',
                        'callback' => 'shariff3uu_share_counts',
                       'permission_callback' => '__return_true',
                        'args'     => array(...

• This reply was modified 3 years, 10 months ago by 3UU.

Hm, zu diesem “W3C-Fehler” stehe ich jetzt vielleicht auf dem Schlacuh, aber die Elemente sind imho ordentlich im fieldset-Block drin. Und werden ja auch genutzt, um nicht plausible Eingaben zu markieren. Und Browser-Unterstuetzung ist laut https://www.w3schools.com/tags/tag_fieldset.asp auch ganz gut. Ich waere da jetzt eigentlich nicht sonderlich motiviert, diese Funktionalitaet rauszuwerfen, nur weil der W3C parser damit nicht klar kommt.

Sorry but I do not really see what problem does this solve??? Please have in mind the we have to find one of the translations supported by the form. And we are looking for the language that the _client_ supports. Because I often have seen browsers in English versions used by staff that does not speak this language… we first try to use GeoIP. If this fails the http_negotiate_language functionality is imho the next best choise.

Version 3.4.0 hat jetzt auch nen RSS-Button.

So, denn externen backend-host gibt es ja nun schion ne Weile. Natuerlich erstmal nur per Konfigurations-Vriable auf dem Backend. Aber ich denke, wenn sich da nicht riesen Badarf kuenftig hier meldet, dann lassen wir das auch erstmal so in ich setze das hier mal auf resolved ??

With version 3.3.2 there is a dynamic generated list of all used custom post types.

Version 3.3.2 mit dem Fix von JP habe ich gerade online gestellt. Probier bitte mal.

I had have spoken with the WP plugin support team and they also can not find these files in the repository. Strange. But if it fixed now I think this was a temporarily problem at the server that we can not solve.

There are no pot-files in the plugin. Please have a look at https://plugins.svn.www.ads-software.com/shariff/trunk/ However I have an idea of what it could be. Do you have an multi-language plugin installed like “CodeStyling Localization” or do you use “WPMU”? These tool try to check all strings in a new/updated plugin and use po/pot-files.

In Version 3.3 wird es die Option fuer eine “exteren Server” geben. Der zieht sich CSS, JS und counter von einem “externen” Server. Auf diesem kannst Du dann auch ein Array mit saemtlichen erlaubten Domains anlegen. Das behebt schonmal den “Wrong Domain”-Fehler.

Wenn Du natuerlich lokal entwickelst, macht es keinen Sinn, alles vom externen Server zu laden ?? Dann einfach in Deiner lokalen Installation das Backend direkt im Shariff-Tag eintragen.

3.3 braucht aber noch was an Tests. Du kannst Dir die Entwicklungsversion aber natuerlich auch jetzt schon von Github https://github.com/3UU/wordpress-shariff-wrapper/ holen.

Oh, I see. I will have a look into it. I think fixed version will be online on Saturday.

Aua, you are right. Not sure why I did not see it at the review ?? However in function

rsearch($folder, $pattern) {

I do not see why

$dir = new RecursiveDirectoryIterator($folder);
$iterator = new RecursiveIteratorIterator($dir);
$files = new RegexIterator($iterator, $pattern, RegexIterator::GET_MATCH);

is needed. Would

$iterator = new RecursiveIteratorIterator($folder)

not work too?

Okay, umso laenger ich drueber gruebel, desto charmanter finde ich die Idee mit einem externen Backend-Server. Vor allem, weil ich auch soein paar “Experten” kenne, die mit Updates bei den Plugins gern mal schlampen ?? Was mich noch stoert:

1.) ist die potenzielle Trennung von JS und Backend im Admin-Bereich macht fuer mich keinen Sinn. Vor allem mit Blick auf durchaus sinnvolle Browser-Features wie sameorigin. Zwei Eingaben sind da imho nur ne ueble Fehlerquelle.

2.) die Eingabe eines Protokolls ist problematisch. Wenn jemand “https://” reinhaut auf einer https-Seite, gibt es auch noch unschoene Warnungen. Daher wuerde ich es lieber mit “example.com/shariff-path” machen und “//” davor sowie “/backend/index.php beziehungsweise /shariff.js” dahinter setzen.

3.) auch der externe Server sollte sich ohne haendische Anpassungen updaten lassen. Also muessen wir die “erlaubten Domains” irgendwo hinterlegen. Eine zusaetzlich Option im Admin-Menue finde ich mit Blick auf all die Leute, die es einfach nur nutzen wollen, irgendwie doof. Ich waere dafuer, dass wir in bewaehrter Weise erstmal wieder mit einem Array in der wp-config.php starten. Das Backend liest die neuerdings ja eh immer aus. Und: Wer einen zentralen Backendserver fuer seine Installationen betreiben will, sollte auch in der Lage sein, nen Array da reinzuschreiben.

Meinungen? JP?

Gerne!!! https://www.ads-software.com/support/view/plugin-reviews/shariff?filter=5 waere nett ??