Opened 12 years ago

Closed 12 years ago

#2773 closed defect (fixed)

[patch][ccla] docparser parses .xd.js files

Reported by: guest Owned by: Neil Roberts
Priority: high Milestone: 1.1
Component: Doc parser Version: 0.4
Keywords: docparser Cc: dante
Blocked By: Blocking:

Description

Currently the API docparser parses all files ending with 'js'. If a cross-domain build has been run previously then there are .xd.js files everywhere which get parsed as well.

patch: 

function getFileList() in docscripts/inc/Dojo.php (~line 53):

  -  if (substr($file, -2)=='js') {
  +  if ((substr($file, -3)=='.js') and (substr($file, -6)!='.xd.js')) {

Change History (5)

comment:1 Changed 12 years ago by guest

grrr, trac keeps stripping off the username. Add robert.coup@… to the CC please.

comment:2 Changed 12 years ago by dante

Cc: dante added; Neil Roberts removed

comment:3 Changed 12 years ago by dylan

Milestone: → 1.1

This is a really easy fix to apply. Let's get it done.

comment:4 Changed 12 years ago by Neil Roberts

(In [12653]) Refs #2773. Update format in prep for real checkin

comment:5 Changed 12 years ago by Neil Roberts

Resolution: → fixed
Status: new → closed

(In [12654]) Fixes #2773. Make sure that .xd.js files don't get picked up during the cron run

Note: See TracTickets for help on using tickets.