hotel-manage-stack/rates
1
0
Fork 0
Code Issues 3 Pull requests Projects Releases Packages Wiki Activity Actions

Fix own hotel excluded from scrape and 1-person rates hidden in matrix

Browse source 
Include tier=own in get_active_hotels() so the own hotel Booking.com
listing is scraped alongside competitors. Change matrix max_persons
filter from hard WHERE to ORDER BY priority so hotels with only
1-person rates (e.g. Old Stocks) still show up rather than being
silently excluded.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
jtricerolph 2026-07-10 00:31:08 +00:00
parent ca76bc2f90
commit dd84ca230a
2 changed files with 10 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

10
backend/api/competitors.py
View file

@ -778,8 +778,9 @@ async def get_competitor_matrix(
WHERE {tier_filter} WHERE {tier_filter}
AND h.is_active = TRUE AND h.is_active = TRUE
AND r.rate_date >= :from_date AND r.rate_date <= :to_date AND r.rate_date >= :from_date AND r.rate_date <= :to_date
AND (r.max_persons IS NULL OR r.max_persons = 2) ORDER BY r.hotel_id, r.rate_date,
ORDER BY r.hotel_id, r.rate_date, r.scraped_at DESC, r.rate_gross ASC NULLS LAST CASE WHEN r.max_persons IS NULL OR r.max_persons = 2 THEN 0 ELSE 1 END,
r.scraped_at DESC, r.rate_gross ASC NULLS LAST
"""), """),
{'from_date': start, 'to_date': end} {'from_date': start, 'to_date': end}
) )
@ -818,8 +819,9 @@ async def get_competitor_matrix(
AND r.rate_date >= :from_date AND r.rate_date <= :to_date AND r.rate_date >= :from_date AND r.rate_date <= :to_date
AND r.availability_status = 'available' AND r.availability_status = 'available'
AND r.rate_gross IS NOT NULL AND r.rate_gross IS NOT NULL
AND (r.max_persons IS NULL OR r.max_persons = 2) ORDER BY r.hotel_id, r.rate_date,
ORDER BY r.hotel_id, r.rate_date, r.scraped_at DESC, r.rate_gross ASC NULLS LAST CASE WHEN r.max_persons IS NULL OR r.max_persons = 2 THEN 0 ELSE 1 END,
r.scraped_at DESC, r.rate_gross ASC NULLS LAST
"""), """),
{'from_date': start, 'to_date': end} {'from_date': start, 'to_date': end}
) )

8
backend/services/booking_scraper.py
View file

@ -295,16 +295,16 @@ def cleanup_stale_batches(db: Session, max_age_minutes: int = 60):
def get_active_hotels(db: Session) -> List[Dict[str, Any]]: def get_active_hotels(db: Session) -> List[Dict[str, Any]]:
"""Return competitor hotels with a booking_com_url (for hotel-page scraping). """Return own + competitor hotels with a booking_com_url (for hotel-page scraping).
Own and market hotels are excluded — market were auto-discovered from search results; Market hotels are excluded — they are auto-discovered from search results and not
own hotel rates come from the Newbook API, not Booking.com scraping.""" individually monitored. Own hotel IS included so its Booking.com listing is tracked."""
rows = db.execute( rows = db.execute(
text(""" text("""
SELECT id, booking_com_id, name, booking_com_url SELECT id, booking_com_id, name, booking_com_url
FROM booking_com_hotels FROM booking_com_hotels
WHERE is_active = TRUE WHERE is_active = TRUE
AND booking_com_url IS NOT NULL AND booking_com_url IS NOT NULL
AND tier = 'competitor' AND tier IN ('own', 'competitor')
ORDER BY display_order, id ORDER BY display_order, id
""") """)
).fetchall() ).fetchall()