Refactor: order and page parameter parsing
This commit is contained in:
		
					parent
					
						
							
								ec9048829f
							
						
					
				
			
			
				commit
				
					
						b12ffac09e
					
				
			
		
					 1 changed files with 9 additions and 10 deletions
				
			
		
							
								
								
									
										19
									
								
								api.php
									
										
									
									
									
								
							
							
						
						
									
										19
									
								
								api.php
									
										
									
									
									
								
							|  | @ -713,11 +713,11 @@ class REST_CRUD_API { | ||||||
| 	} | 	} | ||||||
| 
 | 
 | ||||||
| 	protected function processOrderParameter($order) { | 	protected function processOrderParameter($order) { | ||||||
| 		if ($order) { | 		if (!$order) return false; | ||||||
| 			$order = explode(',',$order,2); | 		$order = explode(',',$order,2); | ||||||
| 			if (count($order)<2) $order[1]='ASC'; | 		if (count($order)<2) $order[1]='ASC'; | ||||||
| 			$order[1] = strtoupper($order[1])=='DESC'?'DESC':'ASC'; | 		if (!strlen($order[0])) return false; | ||||||
| 		} | 		$order[1] = strtoupper($order[1])=='DESC'?'DESC':'ASC'; | ||||||
| 		return $order; | 		return $order; | ||||||
| 	} | 	} | ||||||
| 
 | 
 | ||||||
|  | @ -758,11 +758,10 @@ class REST_CRUD_API { | ||||||
| 	} | 	} | ||||||
| 
 | 
 | ||||||
| 	protected function processPageParameter($page) { | 	protected function processPageParameter($page) { | ||||||
| 		if ($page) { | 		if (!$page) return false; | ||||||
| 			$page = explode(',',$page,2); | 		$page = explode(',',$page,2); | ||||||
| 			if (count($page)<2) $page[1]=20; | 		if (count($page)<2) $page[1]=20; | ||||||
| 			$page[0] = ($page[0]-1)*$page[1]; | 		$page[0] = ($page[0]-1)*$page[1]; | ||||||
| 		} |  | ||||||
| 		return $page; | 		return $page; | ||||||
| 	} | 	} | ||||||
| 
 | 
 | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Maurits van der Schee
				Maurits van der Schee