Project

General

Profile

Actions
Copy link

Bug #5466

closed

Bug #5465: We should not canonify ourselves class context given in ncf

Canonified classes are not properly displayed

Added by Vincent MEMBRÉ over 10 years ago. Updated over 2 years ago.

Status:
Rejected
Priority:
1 (highest)
Assignee:
-
Category:
Web - Technique editor
Target version:
4.1.10
Pull Request:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

We canonify by ourselve classes displayed for each method... and we do not that correctly (see #5465)

Regexp to do this cannot be done easily, so a first step would be to remove that canonification completely

Actions
Copy link
 #1

Updated by Alexis Mousset over 8 years ago

  • Category set to Technique editor - API
Actions
Copy link
 #2

Updated by Alexis Mousset over 7 years ago

  • Status changed from New to Discussion
  • Priority set to 0

I don't understand this ticket, could you clarify where the problem happens?

Actions
Copy link
 #3

Updated by Jonathan CLARKE over 7 years ago

  • Status changed from Discussion to New
Actions
Copy link
 #4

Updated by Jonathan CLARKE over 7 years ago

  • Assignee deleted (Vincent MEMBRÉ)
Actions
Copy link
 #5

Updated by Benoît PECCATTE over 7 years ago

  • Subject changed from Do not display canonified classes to Canonified classes are not properly displayed
  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Priority changed from 0 to 43

Example:
/tmp/${rudder.base[test]}
should be tmp${rudder.base[test]}
bit is tmp${rudder.base[test_}

Actions
Copy link
 #6

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 0.x to 3.1.25
  • Priority changed from 43 to 44
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 3.1.25 to 387
Actions
Copy link
 #8

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 387 to 4.1.10
  • Priority changed from 44 to 45
Actions
Copy link
 #9

Updated by Benoît PECCATTE almost 7 years ago

  • Status changed from New to Rejected

This is done "almost" properly now

Actions
Copy link
 #10

Updated by Benoît PECCATTE almost 7 years ago

Almost: there is no recursion in variable parsing, only 2 level deep.

Actions
Copy link
 #11

Updated by Alexis Mousset over 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - API to Web - Technique editor
  • Priority changed from 45 to 0
Actions
Copy link

Also available in: Atom PDF