kokspflanze / ZfcSitemap

Sitemap for Laminas
MIT License
3 stars 0 forks source link

Not worked out-of-the-box #1

Closed danielgiljara closed 5 years ago

danielgiljara commented 5 years ago

Hello, first of all, I think I'm creating the first issue.

As title says, I've installed the software with composer require and using the option to inject the configuration on my config, it has "buena pinta" (it has appeareance to perform well).

When I go to /sitemap.xml I get the following error page code:

error on line 6 at column 111: xmlParseEntityRef: no name
danielgiljara commented 5 years ago

I think I've found the problem to the solution. I've an & in the title, when is better to have & (for if you want this suggest to improve your software, I not time to PR).

danielgiljara commented 5 years ago

Now I've the error:

error on line 17 at column 92: Opening and ending tag mismatch: link line 0 and head
danielgiljara commented 5 years ago

I think I'm not generating HTML due to standards. Are the author or any other heper interested to catch my case in a peer programming session to incorporate all this casuistics?

kokspflanze commented 5 years ago

hi can you post the complete xml?

normal the xml generation is from zend-navigation, but i will check it, but i need your xml.

on monday i can check it

danielgiljara commented 5 years ago

Hello, how can I get the generated XML from the zend-navigation? I've installed your component again but I don't find in the code where zend-navigation is used.

Monday is a good for me.

El sáb., 28 sept. 2019 a las 15:04, Oliver (notifications@github.com) escribió:

hi can you post the complete xml?

normal the xml generation is from zend-navigation, but i will check it, but i need your xml.

on monday i can check it

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/kokspflanze/ZfcSitemap/issues/1?email_source=notifications&email_token=AAKLZ7NLNRV22A7IUSYOMCDQL5I4FA5CNFSM4I3J5DBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD72ZL7Q#issuecomment-536188414, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKLZ7KVIZZ7NA5CSIYGT43QL5I4FANCNFSM4I3J5DBA .

kokspflanze commented 5 years ago

did you create the navigation with zend-navigation? https://docs.zendframework.com/zend-navigation/quick-start/

i mean the sitemap xml, when you open sitemap.xml

danielgiljara commented 5 years ago

Hi,

No, I'm using the default template provided by zend mvc skeleton.

Maybe should be this? But the first error changed when I modified the title at layout.phtml

I'm writting on mobile.

El dom., 29 sept. 2019 11:44, Oliver notifications@github.com escribió:

did you create the navigation with zend-navigation? https://docs.zendframework.com/zend-navigation/quick-start/

i mean the sitemap xml, when you open sitemap.xml

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/kokspflanze/ZfcSitemap/issues/1?email_source=notifications&email_token=AAKLZ7JRXMINZKKUX4LE6U3QMB2HRA5CNFSM4I3J5DBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD73OSYY#issuecomment-536275299, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKLZ7MSLBTDAU6CDIOKNHTQMB2HRANCNFSM4I3J5DBA .

kokspflanze commented 5 years ago

hi, you have to use zend-navigation, the sitemap will not work without.

danielgiljara commented 5 years ago

Ok, I understand. I think is thing of zend-navigation too as it worked when I changed at & by & amp;</p> <p>For the moment I generated sitemap.xml handly.</p> <p>Thanks for your support. I close the issue.</p> </div> </div> <div class="page-bar-simple"> </div> <div class="footer"> <ul class="body"> <li>© <script> document.write(new Date().getFullYear()) </script> Githubissues.</li> <li>Githubissues is a development platform for aggregating issues.</li> </ul> </div> <script src="https://cdn.jsdelivr.net/npm/jquery@3.5.1/dist/jquery.min.js"></script> <script src="/githubissues/assets/js.js"></script> <script src="/githubissues/assets/markdown.js"></script> <script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.4.0/build/highlight.min.js"></script> <script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.4.0/build/languages/go.min.js"></script> <script> hljs.highlightAll(); </script> </body> </html>