Reply to thread

Message

<blockquote data-quote="arion_p" data-source="post: 528329" data-attributes="member: 45945">Now I understand what you mean. However what you suggest is not possible because javascript calls can be made in numerous attributes of the A tag (e.g. onclick, onmouseover, href, etc). Restricting the search to href only will certainly break some of the existing grabbers. I can only see the following solutions:<ol>
<li data-xf-list-type="ol">Add an option so that a grabber may choose to not search for javascript calls.</li>
<li data-xf-list-type="ol">Limit the search to specific attributes of the A tag (e.g. there is no point in searching the title attribute)</li>
<li data-xf-list-type="ol">Same as 2 but allow the grabber to specify which attribute(s) to search</li>
</ol>Any of the above changes need to be made to the core html parsing code of MP (used also for movie info grabbing and others). As you know we are in feature freeze right now and this change will affect and may break other parts of MP, so it is not possible to be implemented before 1.1.0 release is out.</blockquote>

[QUOTE="arion_p, post: 528329, member: 45945"] Now I understand what you mean. However what you suggest is not possible because javascript calls can be made in numerous attributes of the A tag (e.g. onclick, onmouseover, href, etc). Restricting the search to href only will certainly break some of the existing grabbers. I can only see the following solutions: [LIST=1] [*]Add an option so that a grabber may choose to not search for javascript calls. [*]Limit the search to specific attributes of the A tag (e.g. there is no point in searching the title attribute) [*]Same as 2 but allow the grabber to specify which attribute(s) to search [/LIST] Any of the above changes need to be made to the core html parsing code of MP (used also for movie info grabbing and others). As you know we are in feature freeze right now and this change will affect and may break other parts of MP, so it is not possible to be implemented before 1.1.0 release is out. [/QUOTE]

Verification