LayGridder > Options > LocationForm - Exceeding php memory limit

---

Hi !

LayGridderLocation::get_select_view() is pretty much querying every post on the wp and then inserting it in the DOM as an option for the LayGridder > Options > Location form".

<option value="1">Non classé</option>

a few options laters

<option value="40149">Comment ça marche ?</option>

With 45k posts the page was just failing to display without error bellow 1024Mo allowed to php memory.

So I might be testing this on the wrong type of website for layGridder but even with a few hundred that might be problematic.

@rsepierre

		// get all posts of post type "post"
		$args = array(
			'posts_per_page' => -1,
			'orderby' => 'menu_order',
			'post_type' => 'post'
		);
		$all_posts_query = new WP_Query( $args );

		// get all pages
		$args = array(
			'posts_per_page' => -1,
			'orderby' => 'menu_order',
			'post_type' => 'page'
		);
		$pages_query = new WP_Query( $args );

		// get all posts of custom post types
		$posts_of_custom_post_types = array();
		foreach ($custom_post_types as $post_type) {

			$args = array(
				'posts_per_page' => -1,
				'orderby' => 'menu_order',
				'post_type' => $post_type->name
			);
			$custom_posts_query = new WP_Query( $args );
			if($custom_posts_query->have_posts()){
				$posts_of_custom_post_types[$post_type->name] = $custom_posts_query->posts;
			}
		}

For now I patched every 'post_per_page' => -1 to be 'post_per_page' => 99
(if there is more I probably don't want to target a specific post)

But maybe the best option would be to be able to search when there is more than a defined number of posts

Dear @rsepierre
thank you for this. We will have a look here in the team and will get back to you.

Best!

Marius