Ticket #150 (closed defect: fixed)

Opened 3 years ago

Last modified 22 months ago

Rewrite rule for non-latin characters

Reported by: skorobogatov@… Owned by: mbonetti
Priority: high Milestone: Gregarius 0.5.5
Component: BUGS Version:
Severity: major Keywords:
Cc:

Description

If the feed title contains only non-latin characters rewrite rule convert it into string like this

________________________________________

For Example "Технологии. Тренды. Тенденции." ->

_______________________________________________________

It's not good for non-english users of Gregarius (for me). Can you make the function like WordPress sanitize_title() with hook for plugins?

Change History

Changed 3 years ago by mbonetti

  • status changed from new to assigned

Will take a look at this sanitize_title() thing.

Changed 3 years ago by mbonetti

Fixed in [836]

Changed 3 years ago by mbonetti

  • status changed from assigned to closed
  • resolution set to fixed

Changed 3 years ago by anonymous

  • status changed from closed to reopened
  • resolution fixed deleted

I'm just an amateur, but it seems that changeset 836 doesn't really address the preg_replace issue, there being many more instances where this needs to be patched (i.e. I still get the titles). I think rather than having so many ugly preg_replace thingie, a new function should be called to sanitize titles without treating non-Latin as enemy.

WordPress's sanitize routines seem tedious and complicated, but they do work:

http://trac.wordpress.org/log/trunk/wp-includes/functions-formatting.php

Changed 3 years ago by mbonetti

  • milestone changed from Gregarius 0.5.1 to Gregarius 0.5.5

Changed 2 years ago by sdcosta

Changed 22 months ago by mbonetti

  • status changed from reopened to closed
  • resolution set to fixed

refixed in [1648]

Note: See TracTickets for help on using tickets.