bootstrap3 icon indicating copy to clipboard operation
bootstrap3 copied to clipboard

catalog category page generates error in 3.3.0.8

Open jnugent opened this issue 3 years ago • 2 comments

Hi @NateWr

it looks like the way the master branch is checking for subcategories here:

https://github.com/NateWr/bootstrap3/blob/master/templates/frontend/pages/catalogCategory.tpl#L46

Wasn't included in the tagged release for the version of the theme that gets installed via the plugin gallery for 3.3.0.8. It's still doing:

https://github.com/NateWr/bootstrap3/blob/v3_2_0-4/templates/frontend/pages/catalogCategory.tpl#L46

Which generates a fatal error:

PHP Fatal error: Uncaught Error: Call to a member function wasEmpty() on array

So I think that just needs to be backported back. I upgraded a client from 3.1.2 to 3.3 today and the theme breaks when it is updated via the gallery.

jnugent avatar Feb 15 '22 19:02 jnugent

Arg! Yeah it looks like there are a few commits that aren't in the 3_2_0_4 branch. I was planning to make a release after I got confirmation on this bug fix: https://github.com/NateWr/bootstrap3/issues/177.

I can try to sort this out soon. I need to get this set up with the main/stable-x_x_x branching convention of our other apps.

NateWr avatar Feb 16 '22 10:02 NateWr

Hi @NateWr if it helps, using the main branch for OJS 3.3 seems to resolve the issues. I found another problem earlier this morning with the 3_2_0_4 branch and 3.3 but that goes away with main.

jnugent avatar Feb 16 '22 12:02 jnugent

@asmecher This issue can be closed as resolved in https://github.com/pkp/bootstrap3/pull/172 and confirmed in 3.3/3.4/main.

kaitlinnewson avatar Jun 27 '24 14:06 kaitlinnewson

Closing, thanks, @kaitlinnewson!

asmecher avatar Jun 27 '24 19:06 asmecher