Fix #4855 Ne pas nécessiter la présence d’un htaccess pour traiter l’url d’api de statistiques #4856

Merged
marcimat merged 1 commits from issue_4855_api_statistiques into master 1 month ago
marcimat commented 1 month ago
Owner

2 types d’urls sont créés sur l’api de statistiques,

  • l’une depuis le privé ./transmettre.api/...
  • l’autre depuis le public ../transmettre.api/

J’ai fait là le choix de tout remplacer par ./?action=api_transmettre ou ../?action=api_transmettre mais pour le public on aurait pu remplacer par ../spip.php?action=api_transmettre par exemple.

À voir, mais sinon ça donnait donc cela :

	// Ne pas imposer une redirection d’url
	$url = str_replace(
		[
			'../transmettre.api/',
			'./transmettre.api/',
		],
		[
			'../spip.php?action=api_transmettre&arg=',
			'./?action=api_transmettre&arg=',
		],
		str_replace('?', '&', $url)
	);

J’ai préféré proposer une forme plus simple, je ne crois pas que ça soit très gênant (sauf si le index.php public est modifié peut être ?)

2 types d’urls sont créés sur l’api de statistiques, - l’une depuis le privé `./transmettre.api/...` - l’autre depuis le public `../transmettre.api/` J’ai fait là le choix de tout remplacer par `./?action=api_transmettre` ou `../?action=api_transmettre` mais pour le public on aurait pu remplacer par `../spip.php?action=api_transmettre` par exemple. À voir, mais sinon ça donnait donc cela : ```php // Ne pas imposer une redirection d’url $url = str_replace( [ '../transmettre.api/', './transmettre.api/', ], [ '../spip.php?action=api_transmettre&arg=', './?action=api_transmettre&arg=', ], str_replace('?', '&', $url) ); ``` J’ai préféré proposer une forme plus simple, je ne crois pas que ça soit très gênant (sauf si le index.php public est modifié peut être ?)
marcimat added 1 commit 1 month ago
b_b approved these changes 1 month ago
marcimat added 1 commit 1 month ago
marcimat added 1 commit 1 month ago
marcimat merged commit d96522ee7b into master 1 month ago
marcimat deleted branch issue_4855_api_statistiques 1 month ago

Reviewers

b_b approved these changes 1 month ago
The pull request has been merged as d96522ee7b.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.