samdark / sitemap

Sitemap and sitemap index builder
BSD 3-Clause "New" or "Revised" License
534 stars 86 forks source link

Количество ссылок в карте #65

Open chulcha opened 5 years ago

chulcha commented 5 years ago

при мультиязыковой карте, количество ссылок в Число языков больше

нужно

$this->urlsCount = $this->urlsCount + count($location);

samdark commented 5 years ago

Можно пояснить "количество ссылок в Число языков больше"?

chulcha commented 5 years ago

Если у вас 2 языка, то ссылок в карте будет 2, а не одна

у вас $this->urlsCount число обращений в addItem, а не число переданных в нее ссылок

chulcha commented 5 years ago

Кстати, за этим потянется еще и способ для метода flush, который подразумевает 1 инкремент

$this->urlsCount % $this->bufferSize === 0

samdark commented 5 years ago

https://github.com/samdark/sitemap/pull/66 так?

chulcha commented 5 years ago

Почти if ($this->urlsCount % $this->bufferSize === 0) {

у вас три языка соответственно остаток станет равным нулю при количество языков * bufferSize

Я для этого ввел еще одно свойство для текущего значения буфера

Го это не существенно - 10 или 30

samdark commented 5 years ago

Почему? Мы ведь уже нарастили urlsCount корректно...

chulcha commented 5 years ago

bufferSize = 10 языков = 3

0 + 3 - остаток 3 3+3 - остаток - 3 9+3 - остаток? 2 27+3 = 30 - остаток 0

arvai commented 3 years ago

how do you expect help if you are writing in !english ?

chulcha commented 3 years ago

how do you expect help if you are writing in !english ?

owner reads and writes in russian