Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: allow to use this theme an old php version #737

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: allow to use this theme an old php version #737

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
db33e0e
fix: allow to use this theme an old php version
VinsMach Dec 10, 2024
3baeae2
Update inc/actions.php
VinsMach Dec 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion inc/actions.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,8 @@ function dsi_eventi_filters( $query ) {
*/
function dsi_schede_progetti_filters( WP_Query $query ) {

if ( ! is_admin() && $query->is_main_query() && (is_post_type_archive("scheda_progetto") || (get_queried_object()?->taxonomy ?? null) == "tipologia-progetto") ) {
if ( ! is_admin() && $query->is_main_query() && (is_post_type_archive("scheda_progetto") || is_tax("tipologia-progetto")) ) {
//if ( ! is_admin() && $query->is_main_query() && (is_post_type_archive("scheda_progetto") || (get_queried_object()?->taxonomy ?? null) == "tipologia-progetto") ) {

$orderby = dsi_get_option("ordinamento_progetti", "didattica") ?? 'date';
$order_direction = dsi_get_option("direzione_ordinamento_progetti", "didattica") === 'asc' ? 'asc' : 'desc';
Expand Down
4 changes: 2 additions & 2 deletions inc/breadcrumb.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -641,7 +641,7 @@ protected function add_term_archive_items() {
if ( false !== $taxonomy->rewrite ) {

// If 'with_front' is true, dd $wp_rewrite->front to the trail.
if ( $taxonomy->rewrite['with_front'] && $wp_rewrite->front )
if ( array_key_exists('with_front', $taxonomy->rewrite) && $taxonomy->rewrite['with_front'] && $wp_rewrite->front )
VinsMach marked this conversation as resolved.
Show resolved Hide resolved
$this->add_rewrite_front_items();

// Get parent pages by path if they exist.
Expand Down Expand Up @@ -1112,7 +1112,7 @@ protected function get_post_types_by_slug( $slug ) {

foreach ( $post_types as $type ) {

if ( $slug === $type->has_archive || ( true === $type->has_archive && $slug === $type->rewrite['slug'] ) )
if ( $slug === $type->has_archive || ( true === $type->has_archive && is_array($type->rewrite) && isset($type->rewrite['slug']) && $slug === $type->rewrite['slug']))
Copy link

@enrimk enrimk Dec 18, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non è un'eventualità piuttosto remota?
Cioè, mi pare che rewrite['slug'] qui possa risultare non definito solo nel caso di un post_type con has_archive === true e rewrite === false contemporaneamente (con archivio, ma senza permalink). È così? È un caso che si verifica nell'app?

$return[] = $type;
}

Expand Down
23 changes: 23 additions & 0 deletions inc/utils.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1231,4 +1231,27 @@ function dsi_get_progetti_in_luogo($luogo_id)

return $progetti;
}
}



if(!function_exists("get_page_by_title_new")){
function get_page_by_title_new( $page_title, $output = OBJECT, $post_type = 'page' ) {
$query = new WP_Query([
'post_type' => 'page',
'title' => $page_title,
'post_status' => 'publish', // Opzionale: solo pagine pubblicate
'posts_per_page' => 1, // Opzionale: limitare a un risultato
]);

$page = null;

if ($query->have_posts()) {
$query->the_post();
$page = get_post(); // Recupera l'oggetto del post corrente
wp_reset_postdata(); // Resetta i dati del loop di WordPress
}

return $page;
}
}
Comment on lines +1238 to 1257
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggerirei una formulazione un po' più compatta.

Suggested change
if(!function_exists("get_page_by_title_new")){
function get_page_by_title_new( $page_title, $output = OBJECT, $post_type = 'page' ) {
$query = new WP_Query([
'post_type' => 'page',
'title' => $page_title,
'post_status' => 'publish', // Opzionale: solo pagine pubblicate
'posts_per_page' => 1, // Opzionale: limitare a un risultato
]);
$page = null;
if ($query->have_posts()) {
$query->the_post();
$page = get_post(); // Recupera l'oggetto del post corrente
wp_reset_postdata(); // Resetta i dati del loop di WordPress
}
return $page;
}
}
if (!function_exists("dsi_get_page_by_title")) {
function dsi_get_page_by_title( $page_title ) {
return (new WP_Query([
'post_type' => 'page',
'title' => $page_title,
'post_status' => 'publish',
'posts_per_page' => 1,
]))->post;
}
}