Sluggable _makeUnique is buggy about utility HOT 8 CLOSED

How so? Number 4 wouldn't generate "abc-1" as "abc" doesn't exist, it would stay as "abc".

from utility.

Because you are using LIKE to check the slug.

So the conditions is [slug LIKE] => 'abc%'. Which will return count as 1.

That is how you get 'abc-1' at the Number 4.

Hope it is clear.

from utility.

I'll have to change the LIKE logic then, because I guess "abcd" will also trigger it.

from utility.

Rewrote the queries.

from utility.

This will not work.

For example:

1. abc -> abc
2. abc -> abc-1($count=1)
3. abc -> abc-1($conditions['slug']='abc', $count=1 && $conditions['slug'] = 'LIKE abc-%', $count=1)

You will see point 2 and 3 are producing the same slug.

Why do not you just use a while loop to check the uniqueness of the slug?

from utility.

It runs 2 queries, so if our slug was abc.

1st query - If abc does not exist, then abc is the slug, else...
2nd query - Do a wildcard search for abc-%, which will return slug as abc-#.

The queries aren't doing an &&, they are 2 separate queries.

from utility.

My point 3 is saying:

1st query: abc exists, and $count = 1 , so continue to 2nd query
2nd query: it only check abc-%, which will return $count = 1 again. Which generates a slug with value of 'abc-1'.

You see the problem here? point 2 and 3 are producing the same slug.

from utility.

I just need to fix that if statement, one sec.

from utility.