dynweb

Have you checked that there are no files prepended/appended when rendering the template? Maybe disable Tracy debugger?

Hi Robin,

With the module installed, when I want to edit an image (e.g. crop) in admin, I get the following exception on clicking the "Apply" button:

The exception also occurs if I have previously generated all delayed images with the option "Generate queued variations". The error does NOT occur after uninstalling the module.

Any ideas? Thanks 🙂

Hi,

I'm using your module for the first time, looks really promising 🙂

It seems that password fields *always* show a password toggle, even if the showPasswordToggle() method is not called in user code.

In InputPassword.php, line 105, it unconditionally appends the checkbox:

$this->append($this->showPasswordToggle()->___render());

My solution was the following, but not sure if that won't break other things:

// InputPassword.php

// Line 26
// protected InputCheckbox $showPasswordToggle; // The password toggle checkbox object
protected bool $showPasswordToggle = false;

// Line 85
// $this->showPasswordToggle = $toggle;
$this->showPasswordToggle = true;

// Line 105
// $this->append($this->showPasswordToggle()->___render());
if($this->showPasswordToggle) {
	$this->append($this->showPasswordToggle()->___render());
}

You could add a multi page reference field to the "Project" template to store the associated persons. And synchronize the two reference fields with the excellent "Connect Page Fields" module by Robin S.

https://github.com/Toutouwai/ConnectPageFields

This should help:

I would add a new field with your requirements and migrate the contents from the "old" to the "new" field. Here is how I did it for a migration from a single image ("image") to a multiple images ("images") field:

$gearPages = $pages->find("template=products-view-gear, include=all");
foreach($gearPages as $p) {
    if($p->image) {
        $p->of(false);
        $p->images->add($p->image);
        $p->save();
    }
}

When editing a template like this:

• As long as a template does not define its own access rights, the option "Yes always, regardless of page status or access control" has no effect: it "sticks" in the admin interface, but the folder /site/access/files/XXX is not prefixed with a "-" and access control is not enforced. The file is delivered by Apache.
• If the template defines its own access right, it works as expected (by me). The file is delivered by PHP.

It this expected behavior? Is there some good reason why a template must define its own access rights to enforce access control to files?

Thank you for your insights.

Edit: Tested on latest PW stable, pristine install.

1 hour ago, LMD said:

Does Stripe allow sellers to charge the buyer's card when items are shipped (which could be several days/a week later) instead of immediately when an order is placed?

You can choose on your implementation if you want to charge the user's card immediately ("capture") or only reserve the funds for later capturing (via Stripe admin or API).

The only problem is that it doesn't make much of a difference to the user (Stripe event advises that many users won't see a difference on their CC statement between "reserved" and "captured" amounts). Even if an amount is "only" reserved, the client can no longer dispose of these funds.

I would definitely be interested... if some client would ask for a (PW integrated) forum. Must be at least 10 years now that I haven't had such a requirement.

So I am not sure that there still exists a sizable market for (new) forum softwares...

4 hours ago, adrian said:

I like the idea, but the problem is that sometimes it seems like it's possible for Tracy to double load without the composer files, which is why this line was added in the first place. Maybe I need to install nette via composer myself and see if I can reproduce and come up with a better check.

if(class_exists('\Tracy\Debugger', false) && Debugger::isEnabled()) return;

Works for me. We just tell PHP to skip autoloading, so it will never know about the composer version.

2 hours ago, adrian said:

if(class_exists('\Tracy\Debugger')) return;

Does that work for you with nette included via composer?

Not really ? At this point, Debugger is probably not enabled, so no output at all.

if(class_exists('\Tracy\Debugger')) {
	\Tracy\Debugger::enable();
	return;
};

This one works, but the Tracy bar has none of the ProcessWire specific info (as the init() method returns early).

if(class_exists('\Tracy\Debugger')) {
	$this->wire('session')->error('Cannot activate TracyDebugger because another version is already installed, maybe via composer?');
	return;
};

This kind of error message, however, would be really helpful for people like me who don't remember the stuff they have in their vendor directory...

20 minutes ago, adrian said:

It's still weird to me though because that new line I added should prevent Tracy being loaded a second time. Do you need that composer include, or can you remove it without affecting anything else?

Seems as if Debugger::isEnabled() returns false after a fresh composer install so it can't prevent double loading.

I just have to be careful not to require tracy/tracy via composer (or nette/nette that adds *all* packages), so this shouldn't be a major problem ? 

OK, problem solved ?

Tracy is working with a pristine PW install.

All my "customized" installs have a vendor directory where I (previously) did a

composer require nette/nette

which also installs tracy in the vendor dir. And this version is loaded first, resulting in the "Cannot declare..." error. Thank you for your help, @adrian !

Thank you, @adrian

I can reproduce the error on my local dev server and on the live server(s).

I already tried to to refresh modules and to clean the file compiler cache, to no avail.

I still have to test it with a pristine processwire install, I will let you know...

Starting with version 4.3.26, I get a compile error after installing/upgrading Tracy:

Compile Error: Cannot declare interface Tracy\ILogger, because the name is already in use (line 16 of site/modules/TracyDebugger/tracy-2.9.x/src/Tracy/Logger/ILogger.php)

It seems that this commit https://github.com/adrianbj/TracyDebugger/commit/10e1d3a28a2719d120325eaf4559505c3ebd98a9 is source of the error. When removing the line

if(class_exists('\Tracy\Debugger') && Debugger::isEnabled()) return;

everything is working again.

Tested on PW 3.0.184 and 3.0.200. 

Any input will be appreciated -- I have no idea where to start ?

Download works for me, too.

Just in case: Have you checked that you have the correct time on your localhost? That might prevent the cert to validate.

You might want to try

$http->download($url, $file, ['verify_peer' => false]);

Or maybe additional context options: https://www.php.net/manual/en/context.ssl.php 

Sorry to derail this thread, I would like to download the new version from my download link, but it seems to be the same version that I bought in May 2020. It always says v0.1.2 β...

Thank you ?

6 hours ago, fruid said:

it's already inside a repeater-field so it would be a repeater field inside a repeater-field :-/

This shouldn't be a problem ?

Have you ever thought about using a repeater field (instead of CKE) for user input? One repeater item for each tab? This would be much easier for the client and certainly less error prone... 

57 minutes ago, teppo said:

when a page is indexed, output formatting should be disabled, and thus this sort of problem shouldn't happen. ?

This put me on the right track ? I had a Pages::saved hook that set output formatting to true (when it shouldn't). Thank you!

1 hour ago, teppo said:

Actually it sounds like your search_index field could be single-value textarea, instead of textarea (multi-language). If so, you'll have to change the type of the field manually.

It is multi-language.

As far as I understand, $page->get($field) returns the field value (a string) if output formatting is true, and a field object if output formatting is false (what we need here).

Using the module on a multi-language site. When saving a page, I get the following error: "Call to a member function setLanguageValue() on string", File: .../modules/SearchEngine/lib/Indexer.php:99

 98:                        if ($index_field_exists) {
 99:                            $page->get($index_field)->setLanguageValue($language, $index[$language->id]);
101:                        }

I suppose we need a $page->of(false) here before calling setLanguageValue() ?

2 hours ago, kongondo said:

Yes. The value of PHP (php.ini) post_max_size was low.

Same for me ? Issue solved, thank you.

Hi @kongondo,

On 2/27/2020 at 2:43 PM, Stefanowitsch said:

I just installed the Media Manager on a clean PW installation and encountered a problem: Multiple image upload is not working.

I am unter the "upload" tab and drag some images into the field. Then click "start". The upload progress bar is growing for 1 second then suddenly it shrinks back to zero and the image upload list is cleared. No images were uploaded. What is wrong?

I'm encountering the same problem as @Stefanowitsch on the latest Media Manager: Multiple image uploads not working, exactly how he described it. Did you find a solution?

Quote

Second - whenever I alter something in the "settings" tab it is NOT saved. For example: Upload mode -> Single file uploads set to false.

I do *not* experience this problem, so it may be unrelated...

Thank you ?